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.

Sign up for GitHub

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

Jump to bottom

GPU installation update for SCICO #334

Closed
FernandoDavis opened this issue Sep 6, 2022 · 5 comments · Fixed by #332
Closed

GPU installation update for SCICO #334

FernandoDavis opened this issue Sep 6, 2022 · 5 comments · Fixed by #332
Assignees
@FernandoDavis
Labels
documentation Improvements or additions to documentation

Comments

@FernandoDavis
Copy link
Contributor

FernandoDavis commented Sep 6, 2022 •
edited
Loading

While helping Maliha, from the Imaging Lab @ Purdue, install SCICO with GPU today I found the following error in the guide:

image

It says the following:

image

To fix it all I did was update:

pip install --upgrade "jaxlib==0.3.0+cuda11.cudnn805" -f https://storage.googleapis.com/jax-releases/jax_releases.html

to:

pip install "jaxlib==0.3.0+cuda11.cudnn805" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html

This was the result:

image
@FernandoDavis FernandoDavis self-assigned this Sep 6, 2022
@bwohlberg
Copy link
Collaborator

The significant change is in the URL for the packages? Does this still work if we keep the --upgrade argument to pip?

@bwohlberg bwohlberg added the documentation Improvements or additions to documentation label Sep 7, 2022
@FernandoDavis
Copy link
Contributor Author

Yeah, it was something minor, I can re-do the installation and test if the upgrade argument is still fine.

@FernandoDavis
Copy link
Contributor Author

Works just fine with the --upgrade argument as long as cuda is added to the url! So it should be an easy fix to the install doc.

@bwohlberg
Copy link
Collaborator

Thanks for checking. I'll make the change to the docs.

bwohlberg added a commit that referenced this issue Sep 7, 2022
@bwohlberg
Rephrase some text and address #334
eb62125
@bwohlberg bwohlberg linked a pull request Sep 7, 2022 that will close this issue
Significant docs extension and restructuring #332
Merged
@FernandoDavis FernandoDavis mentioned this issue Sep 7, 2022
Closed
@FernandoDavis
Copy link
Contributor Author

Ah ill delete the other PR, I think we did it at the same time haha.

bwohlberg added a commit that referenced this issue Sep 9, 2022
@bwohlberg
Significant docs extension and restructuring (#332)
360b670 
* Start work on overview edits

* Work in progress on docs restructure

* Fix residual md formatting and add missing references

* Some markup improvements

* Update submodule

* Trivial edits

* Various improvements

* Various improvements

* Restucture amd rename Important Classes section

* Change section title

* Minor change

* Use more standard header level indicators

* Move developers/contributors page

* Rephrase some text and address #334

* Correct chapter number

* Add note on limitations of non-JAX packages

* Bibtex updates

* Change header symbol

* Move external package notes to Notes section and restructure that section
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@FernandoDavis FernandoDavis

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Significant docs extension and restructuring lanl/scico
Fixed installation for GPU. lanl/scico
2 participants
@bwohlberg @FernandoDavis