JuliaGPU / Adapt.jl

Other
86 stars 24 forks source link

fix adapt for reinterpret(reshape, ...) #39

Closed simeonschaub closed 3 years ago

simeonschaub commented 3 years ago

This was added in JuliaLang/julia#37559 and is currently not handled correctly by Adapt.

codecov[bot] commented 3 years ago

Codecov Report

Merging #39 (567712d) into master (c7dad7f) will increase coverage by 1.29%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
+ Coverage   71.42%   72.72%   +1.29%     
==========================================
  Files           3        3              
  Lines          42       44       +2     
==========================================
+ Hits           30       32       +2     
  Misses         12       12              
Impacted Files Coverage Δ
src/wrappers.jl 68.96% <100.00%> (+2.29%) :arrow_up:

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 c7dad7f...567712d. Read the comment docs.

simeonschaub commented 3 years ago

Bump. Would it be possible to also tag a release right away?

maleadt commented 3 years ago

Sure. Thanks!