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

fix: CUDA package optional for FluxMPIExt #2488

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

askorupka
Copy link
Collaborator

@askorupka askorupka commented Oct 4, 2024

Hotfix for #2464 the issue reported by @kishore-nori (#2464 (comment))

@CarloLucibello CarloLucibello merged commit ef19396 into master Oct 4, 2024
6 of 9 checks passed
@kishore-nori
Copy link

Thank you very much @askorupka

@askorupka
Copy link
Collaborator Author

You're welcome and thanks for reporting the bug @kishore-nori !

@kishore-nori
Copy link

Hi @askorupka, sorry for bugging you again on this. I have come across the same precompilation error again when CUDA.jl is in the package environment. And I add MPI.jl and Flux.jl. (The error doesn't occur when CUDA.jl is not in the package environment)

ERROR: LoadError: ArgumentError: Package FluxMPIExt does not have CUDA in its dependencies:
- You may have a partially installed environment. Try `Pkg.instantiate()`
  to ensure all packages in the environment are installed.
- Or, if you have FluxMPIExt checked out for development and have
  added CUDA as a dependency but haven't updated your primary
  environment's manifest file, try `Pkg.resolve()`.
- Otherwise you may need to report an issue with FluxMPIExt 

can you check if this occurs for you as well? I would suggest clearing the Flux.jl compilations from .julia folder, to reproduce the error.

@ToucheSir
Copy link
Member

I would recommend continuing discussion on #2490 since open issues are more discoverable.

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.

4 participants