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

Update numba if NVIDIA_PYTORCH_VERSION not correct #3614

Merged
merged 2 commits into from
Feb 8, 2022
Merged

Update numba if NVIDIA_PYTORCH_VERSION not correct #3614

merged 2 commits into from
Feb 8, 2022

Conversation

itzsimpl
Copy link
Contributor

@itzsimpl itzsimpl commented Feb 5, 2022

Update/install numba if NVIDIA_PYTORCH_VERSION is not equal to the one in Dockerfile. Resolves #3613

Signed-off-by: Iztok Lebar Bajec [email protected]

itzsimpl and others added 2 commits February 5, 2022 22:41
Update/install numba if `NVIDIA_PYTORCH_VERSION` is not equal to the one in `Dockerfile`. Resolves #3613  

Signed-off-by: Iztok Lebar Bajec <[email protected]>
Copy link
Collaborator

@titu1994 titu1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering if Numba should be commented out for the time being because of it's incompatibility with Numpy. Pushing it to the top just means it needs to be installed before Nemo itself now. Well, that is fine for now but I will revert this when the next Numba rc version is released.

@titu1994 titu1994 merged commit c37cee0 into NVIDIA:main Feb 8, 2022
fayejf pushed a commit that referenced this pull request Mar 2, 2022
Update/install numba if `NVIDIA_PYTORCH_VERSION` is not equal to the one in `Dockerfile`. Resolves #3613  

Signed-off-by: Iztok Lebar Bajec <[email protected]>

Co-authored-by: Oleksii Kuchaiev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

./reinstall.sh crashes due to not being able to uninstall llvmlite
3 participants