Skip to content

bump version to v0.8.0 #3432

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
Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

bump version to v0.8.0 #3432

wants to merge 10 commits into from
+137 −51

Conversation

lvhan028
Copy link
Collaborator

@lvhan028 lvhan028 commented Apr 14, 2025 •
edited
Loading

Hi, @jinminxi104
#3420 and #3436 are cherry-picked commits from dev branch. Please help review.

@lvhan028 lvhan028 requested review from jinminxi104 and irexyc April 17, 2025 07:48
@irexyc
Copy link
Collaborator

irexyc commented Apr 17, 2025 •
edited
Loading

I think we should add chat-template matching strategy for internvl3 series .

@lvhan028
Copy link
Collaborator Author

lvhan028 commented Apr 17, 2025 •
edited
Loading

I think we should add chat-template matching strategy for internvl3 series .

That's a great suggestion. I'll drop another PR to fix it

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

@irexyc irexyc irexyc approved these changes

@jinminxi104 jinminxi104 Awaiting requested review from jinminxi104

At least 2 approving reviews are required to merge this pull request.

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

Successfully merging this pull request may close these issues.
5 participants
@lvhan028 @irexyc @zhulinJulia24 @yao-fengchen @DoorKickers