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

Gemini chat wrapper fix #233

Merged
merged 35 commits into from
May 24, 2024
Merged

Conversation

zyzhang1130
Copy link
Contributor

@zyzhang1130 zyzhang1130 commented May 17, 2024


name: Pull Request
about: Create a pull request

Description

fixed the error arising when response does not contain any text due to the reasons flagged by Google. One such case is when running the werewolf example with Gemini chat.

Checklist

Please check the following items before code is ready to be reviewed.

  • Code has passed all tests
  • Docstrings have been added/updated in Google Style
  • Documentation has been updated
  • Code is ready for review

zyzhang1130 and others added 23 commits April 19, 2024 14:45
…d local hugging face model and finetune loaded model with hugging face dataset

Added features to download models from hugging face model hub/load local hugging face model and finetune loaded model with hugging face dataset. Model loading and fine-tuning can happen both at the initialization stage and after the agent has been initialized (see README in `agentscope/examples/load_finetune_huggingface_model` for details). Major changes to the repo include creating the example script `load_finetune_huggingface_model`, adding a new model wrapper `HuggingFaceWrapper`, and creating a new agent type Finetune_DialogAgent. All changes are done in a new example directory `agentscope/examples/load_finetune_huggingface_model`.
made customized hyperparameters specification available from `model_configs` for fine-tuning at initialization, or through `fine_tune_config` in `Finetune_DialogAgent`'s `fine_tune` method after initialization
fixed issue related to `format` method
updated the dependencies needed
fixed the issue arisen when `response` object doesn't contain are text due to reasons flagged by Google
Copy link
Collaborator

@garyzhang99 garyzhang99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be possible for you to exclude the changes that are not related to the Gemini Wrapper from this PR? The SFT-related modifications seem to be intertwined with those for the Gemini Wrapper, and it might be advisable for a PR to concentrate mainly on a single alteration.

@zyzhang1130
Copy link
Contributor Author

Would it be possible for you to exclude the changes that are not related to the Gemini Wrapper from this PR? The SFT-related modifications seem to be intertwined with those for the Gemini Wrapper, and it might be advisable for a PR to concentrate mainly on a single alteration.

Changes made to conversation_with_agent_with_finetuned_model is now decoupled from this branch.

@DavdGao DavdGao linked an issue May 24, 2024 that may be closed by this pull request
Copy link
Collaborator

@DavdGao DavdGao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@DavdGao DavdGao merged commit 5d7638d into modelscope:main May 24, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue with test_gemini_chat (gemini_test.GeminiModelWrapperTest)
3 participants