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

for Julia>=1.7 CUDA no longer exports @atomic #36

Merged
merged 1 commit into from
Sep 9, 2021

Conversation

gaelforget
Copy link
Member

With Julia v1.7 I could no longer compile PlanktonIndividuals.jl which has to do with JuliaGPU/CUDA.jl#1097

explicitly calling CUDA.@atomic fixes the issue I believe

@codecov-commenter
Copy link

Codecov Report

Merging #36 (d68cfa0) into master (712b6b1) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #36   +/-   ##
=======================================
  Coverage   72.87%   72.87%           
=======================================
  Files          37       37           
  Lines        1589     1589           
=======================================
  Hits         1158     1158           
  Misses        431      431           
Impacted Files Coverage Δ
src/output/diagnostics_plankton.jl 60.97% <0.00%> (ø)
src/plankton/physiology/counts.jl 36.98% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 712b6b1...d68cfa0. Read the comment docs.

Copy link
Member

@zhenwu0728 zhenwu0728 left a comment

Choose a reason for hiding this comment

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

Hi Gael, Thanks for bringing this up!

@zhenwu0728 zhenwu0728 merged commit 6a9fe59 into JuliaOcean:master Sep 9, 2021
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.

3 participants