Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: correct the json error for gpt4 #171

Merged
merged 1 commit into from
Jul 12, 2024

Conversation

Zhang-Henry
Copy link
Contributor

Thank you for your contribution to AIOS!
Before submitting the pull request, please ensure the PR meets the following criteria! This helps improve the efficiency of the review process.

Code Quality

Before submitting your PR, you need to follow the steps below to maintain code quality.

  • Use pip install -r requirements-dev.txt to install the extra dependencies in requirements-dev.txt for the following checks.
  • Use pre-commit install to install pre-commit locally before you commit messages. The pre-commit can help correct the style that are added or modified.

PR title and classification

Only specific types of PRs will be reviewed. The PR title is prefixed appropriately (i.e., "prefix: description") to indicate the type of change. Please use one of the prefixs as below:

  • feat Add new features
  • fix Fix bugs
  • docs Modify documents like README, CONTRIBUTE
  • style Modify code format like space and comma without changing code logic
  • refactor Refactor code structure without adding new features or fixing new bugs
  • perf Improve performance or user experience
  • test Test features, including unit test and integration test
  • chore Change the build procedure or add dependencies
  • revert Revert to the previous version

PR messages

  • Description: a description of the change
  • Issue: the issue # it fixes, if applicable
  • Dependencies: any dependencies required for this change

For the Reviews

@dongyuanjushi dongyuanjushi added the fix bug fix bugs label Jul 12, 2024
@dongyuanjushi dongyuanjushi merged commit 4a75748 into agiresearch:main Jul 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix bug fix bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants