-
Notifications
You must be signed in to change notification settings - Fork 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
[RPD-277] [BUG] Fix setup.sh
for the recommendation example
#34
[RPD-277] [BUG] Fix setup.sh
for the recommendation example
#34
Conversation
…nanconda scipy wheels
.github/workflows/ci-llm.yml
Outdated
sed '/zenml/d' llm/requirements.txt > llm/requirements2.txt | ||
mv llm/requirements2.txt llm/requirements.txt |
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.
zenml
has been removed from requirements so this is probably not needed.
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.
LGTM
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.
LGTM
scikit-surprise
does not provide wheels for installation. Therefore when using apyproject.toml
based virtual environment it will not be able to installscikit-surprise
due to not adhering to PEP 517 installing standards. This updates the requirements to install thescikit-surprise
library differently.This PR also updates the CI to fix a bug introduced by Cython3 with pyyaml, described here: yaml/pyyaml#724