-
Notifications
You must be signed in to change notification settings - Fork 53
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
merge code local_evaluator.py #242
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
CMZSrost
commented
Jan 31, 2024
- Description: merge code into local_evaluator,
- Issue: the issue release 0.3.0 #235 ,
- Dependencies: ,
- Tag maintainer: @stonekim, @danielhjz, @ZingLix @Dobiichi-Origami.
fix ckbk local_eval_with_qianfan.ipynb
LGTM |
* fix prompt render * revert format * enable negative prompt for non txt2image
* add new api * 1 * add doc * remove --------- Co-authored-by: root <[email protected]>
stonekim
reviewed
Feb 2, 2024
* fix stream access token expired * fix async access token
* support qianfan plugin in client * fix lint * support extra_parameters * add doc * fix lint * optimize input
* feat: mv src to python * init go * optimize * support stream * optimize code * optimize code * add ut * add ut * add ci * update go version * update ci * fix mock server * update script * update script * fix makefile * update script * update script * fix * optimize setting * fix * compat * fix lint * add python version * fix ci * fix script * fix script * update script * fix script * fix script * fix script * fix script * fix script * fix script * update version * add ut * add lint * fix ci * fix folder * update readme && move doc template * update readme * refactor * refactor option * test * test * test * test * fix lint * rename * refactor * reset * fix script * update readme * fix lint * add comment * add api error * change go version * add close in body * update workflow * Go (#4) * feat: mv src to python * init go * optimize * support stream * optimize code * optimize code * add ut * add ut * add ci * update go version * update ci * fix mock server * update script * update script * fix makefile * update script * update script * fix * optimize setting * fix * compat * fix lint * add python version * fix ci * fix script * fix script * update script * fix script * fix script * fix script * fix script * fix script * fix script * update version * add ut * add lint * fix ci * fix folder * update readme && move doc template * update readme * refactor * refactor option * test * test * test * test * fix lint * rename * Multi language go (#3) * fix: dataset, trainer, evaluaton bug fix (#237) * fix * add more import on __init__.py in dataset package * cookbook update * refactor * reset * fix script * update readme * fix lint * add comment * add api error * change go version * add close in body * update workflow --------- Co-authored-by: Dobiichi-Origami <[email protected]> --------- Co-authored-by: shikuan <[email protected]> Co-authored-by: Dobiichi-Origami <[email protected]> * fix workflow * add cache to actions * fix workflow * update workflow * update workflow * remove changelog * fix rtd * add go release action * add copyright --------- Co-authored-by: shikuan <[email protected]> Co-authored-by: Dobiichi-Origami <[email protected]>
* fix * add more import on __init__.py in dataset package * cookbook update * improve local evaluation * format * add eval local function * add comment * fix unit test * add * small optimization * add subtitle * fix * fix thread work pool concurrent bug
fix ckbk local_eval_with_qianfan.ipynb
…alEval_code # Conflicts: # python/qianfan/evaluation/local_evaluator.py
danielhjz
approved these changes
Feb 2, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.