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 Baichuan2 prompt format #10334

Merged
merged 5 commits into from
Mar 19, 2024
Merged

Fix Baichuan2 prompt format #10334

merged 5 commits into from
Mar 19, 2024

Conversation

NovTi
Copy link
Contributor

@NovTi NovTi commented Mar 6, 2024

Description

Fix Baichuan2 prompt format referred from link1 and link2.

Copy link
Contributor

@hkvision hkvision left a comment

Choose a reason for hiding this comment

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

Also need to update the results in the README?

@hkvision hkvision requested a review from plusbang March 7, 2024 06:08
Copy link
Contributor

@plusbang plusbang left a comment

Choose a reason for hiding this comment

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

@NovTi
Copy link
Contributor Author

NovTi commented Mar 8, 2024

Please also update the prompt format in https://github.com/intel-analytics/BigDL/blob/main/python/llm/example/GPU/PyTorch-Models/Model/baichuan2/generate.py#L25

Also need to update the results in the README?

Have updated README and PyTorch-Models/Model/baichuan2/generate.py

Comment on lines 27 to 28
# Though this is the official prompt format, we found it has problem generating English answers
# If you want to ask English questions, we recommend you to change the prompt format
Copy link
Contributor

Choose a reason for hiding this comment

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

For English prompt, you are recommended to change the prompt format.

@hkvision hkvision merged commit effb9aa into intel-analytics:main Mar 19, 2024
27 checks passed
liu-shaojun pushed a commit that referenced this pull request Mar 25, 2024
* Fix Baichuan2 prompt format

* Fix Baichuan2 README

* Change baichuan2 prompt info

* Change baichuan2 prompt info
@NovTi NovTi deleted the baichuan branch March 28, 2024 02:37
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.

None yet

3 participants