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.

Sign up for GitHub

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

Jump to bottom

[Feature] Support ChatGLM3-6B #222

Merged
merged 6 commits into from
Nov 14, 2023
Merged

[Feature] Support ChatGLM3-6B #222

merged 6 commits into from
Nov 14, 2023

Conversation

LZHgrla
Copy link
Collaborator

@LZHgrla LZHgrla commented Nov 13, 2023

  1. set bos_token_id to [64790, 64792] for ChatGLMTokenizer in encode_fn
  2. set encode_special_tokens=True for ChatGLM3 tokenizer

@pppppM pppppM merged commit bda70b1 into InternLM:main Nov 14, 2023
1 check passed
llkn-2 pushed a commit to llkn-2/xtuner that referenced this pull request Jul 31, 2024
@LZHgrla
[Feature] Support ChatGLM3-6B (InternLM#222)
d432377 
* add chatglm cfgs

* add chatglm3 template

* fix bos_token_id bug

* add encode_special_tokens

* update readme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pppppM pppppM pppppM approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LZHgrla @pppppM