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

llama : fix codeshell support (#8250) #8599

Merged
merged 4 commits into from
Jul 22, 2024

Conversation

hankeke303
Copy link
Contributor

@hankeke303 hankeke303 commented Jul 20, 2024

Hi, @ggerganov

We submitted support for CodeShell in PR #5016, but with the update of llama.cpp, there were some issues with CodeShell support (We have reported this in Issue #8250). This PR aims to sync with the latest version of the repository and fix these issues.

@github-actions github-actions bot added the python python script changes label Jul 20, 2024
@ggerganov
Copy link
Owner

After resolving the conflict from #8609, we can merge

@hankeke303
Copy link
Contributor Author

hankeke303 commented Jul 22, 2024

After resolving the conflict from #8609, we can merge

@ggerganov We have resolve this conflict. Please review and approve it again.

Thanks for taking some time to review it!

src/llama.cpp Outdated Show resolved Hide resolved
src/llama.cpp Outdated Show resolved Hide resolved
@hankeke303
Copy link
Contributor Author

@ggerganov Sorry for not noticing the order before, we have corrected it.

@ggerganov ggerganov merged commit 081fe43 into ggerganov:master Jul 22, 2024
1 check passed
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Jul 27, 2024
* llama : fix codeshell support

* llama : move codeshell after smollm below to respect the enum order
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python python script changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants