-
-
Notifications
You must be signed in to change notification settings - Fork 609
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
make gpu(x) = gpu_device()(x)
#2502
Conversation
9562811
to
fa167e4
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2502 +/- ##
==========================================
- Coverage 34.58% 33.46% -1.13%
==========================================
Files 35 31 -4
Lines 1995 1829 -166
==========================================
- Hits 690 612 -78
+ Misses 1305 1217 -88 ☔ View full report in Codecov by Sentry. |
The tests I temporarily marked as broken should be fixed by LuxDL/MLDataDevices.jl#88 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me (once broken tests are working again), especially since conv cpu-gpu transfer for AMD GPU is working again.
ops, should have waited LuxDL/MLDataDevices.jl#88 for unbreaking the tests. I'll wait for LuxDL/MLDataDevices.jl#88 before tagging. |
As a follow-up to #2492, this removes every residual implementation for data movement between devices in favor of relying entirely on MLDataUtils.jl. The 2 PRs combined save us ~700 lines of code while also extending functionalities.
After this,
gpu
andcpu
just create agpu_device
orcpu_device
on the fly and apply it to the argument:This also fixes the problem with transferring conv layers to amdgpu devices discussed in #2498 (comment)
TODO