-
Notifications
You must be signed in to change notification settings - Fork 63
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
Update runtime's python packages #706
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,24 +1,24 @@ | ||
# Copied from: https://github.com/elyra-ai/elyra/blob/main/etc/generic/requirements-elyra.txt | ||
|
||
# This is a comprehensive list of python dependencies that Elyra requires to execute Jupyter notebooks. | ||
ipykernel==6.13.0 | ||
ipython==8.10.0 | ||
ipykernel==6.29.5 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @harshad16 I've manually updated this file to the latest versions, and I've applied the same versions to the Pipfiles example as well. The packages on the requirments.txt on elyra repo looks old: https://github.com/elyra-ai/elyra/blob/main/etc/generic/requirements-elyra.txt Since I wasn't certain which versions to use, I opted to go with the latest updates. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Personally I would remove any changes to these There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. All the changes look good. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Tnx for pointing this out, the changes have been incorporated under this commit 664ea37 |
||
ipython==8.27.0 | ||
ipython-genutils==0.2.0 | ||
jinja2==3.1.4 | ||
jupyter-client==7.3.1 | ||
jupyter-core==4.11.2 | ||
MarkupSafe==2.1.1 | ||
minio==7.1.15 | ||
nbclient==0.6.3 | ||
nbconvert==7.1.0 | ||
nbformat==5.4.0 | ||
papermill==2.3.4 | ||
pyzmq==24.0.1 | ||
prompt-toolkit==3.0.30 | ||
requests==2.31.0 | ||
tornado==6.3.3 | ||
traitlets==5.1.1 | ||
urllib3==1.26.9 | ||
jupyter-client==8.6.3 | ||
jupyter-core==5.7.2 | ||
MarkupSafe==2.1.5 | ||
minio==7.2.8 | ||
nbclient==0.10.0 | ||
nbconvert==7.16.4 | ||
nbformat==5.10.4 | ||
papermill==2.6.0 | ||
pyzmq==26.2.0 | ||
prompt-toolkit==3.0.47 | ||
requests==2.32.3 | ||
tornado==6.4.1 | ||
traitlets==5.14.3 | ||
urllib3==2.2.3 | ||
# | ||
# These excluded are transitive dependencies of the included python packages. | ||
#ansiwrap==0.8.4 | ||
|
Large diffs are not rendered by default.
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.
Is this version decrease intentional? 🤔 (same applies for this change in the rest of the files)
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.
Yeah, because on jupyter ds notebook we have
onnxconverter-common = "~=1.13.0"
I kept the same on the runtime as wellThere 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.
I understand the reason for the onnxconverter-common - but my question was for the codeflare-sdk.
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.
I looked here - so I guess you tried to aligned these images... well, it's probably not a big deal for now as the regular codeflare-sdk update should fix it for all the images. But we should be sure we'll run it explicitly before the release then...