-
Notifications
You must be signed in to change notification settings - Fork 0
PKG-5550: Add gguf package & Update to 0.9.1 #1
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
Conversation
Linter check found the following problems:ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_e5fmpk5tml/clone` failed. (See above for error) The following problems have been found:===== WARNINGS =====
===== ERRORS =====
|
@jnoller You will need to create a file named upload_channels:
- ai-staging You can add it into this PR. |
@JeanChristopheMorinPerso Added abs.yaml to the PR |
Linter check found the following problems:ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_0cr6exo34o/clone` failed. (See above for error) The following problems have been found:===== WARNINGS =====
===== ERRORS =====
|
Linter check found the following problems:ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_ffpap9l4ao/clone` failed. (See above for error) The following problems have been found:===== WARNINGS =====
|
Co-authored-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>
Linter check found the following problems:ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_e380gbzfaw/clone` failed. (See above for error) The following problems have been found:===== ERRORS =====
|
Co-authored-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>
Co-authored-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>
Co-authored-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the missing dependency (unless I am looking in the wrong place). Other than that looks good.
@JeanChristopheMorinPerso @Jrice1317 @ryanskeith please re-review with the latest changes. I added pyyaml, but also moved to using the llama.cpp project's releases directly - the pypi project is pretty out of date and to support adding the llama-cpp-scripts package it makes sense to pin all three (include llama.cpp-feedstock) the same way. The llama-cpp-scripts pyproject.toml in the root of the llama.cpp tree has always had a version of 0.0.0 but it has a dependency on a matching version of the gguf package. This way all 3 packages work from the source repo directly for now. |
I'm closing this PR: After working on packaging the llama-cpp-scripts work and working through the dependency tree, I think it makes more sense collapse the |
gguf 0.9.1
Destination channel: ai-staging
Links
Explanation of changes:
sentencepiece
dependency