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

Preserve the buffer type when broadcasting. #2203

Merged
merged 2 commits into from
Dec 19, 2023
Merged

Conversation

maleadt
Copy link
Member

@maleadt maleadt commented Dec 13, 2023

Fixes #2191.
Depends on JuliaGPU/Adapt.jl#75 and JuliaGPU/GPUArrays.jl#506.
Will also require a KernelAbstractions.jl compat bump, once the Adapt PR is merged and tagged.

See JuliaGPU/GPUArrays.jl#506 for explanation.

@maleadt maleadt marked this pull request as ready for review December 19, 2023 13:34
Copy link

codecov bot commented Dec 19, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (47168b7) 72.92% compared to head (9f1b01d) 61.53%.

Files Patch % Lines
src/array.jl 66.66% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #2203       +/-   ##
===========================================
- Coverage   72.92%   61.53%   -11.39%     
===========================================
  Files         159      154        -5     
  Lines       14654    14092      -562     
===========================================
- Hits        10686     8672     -2014     
- Misses       3968     5420     +1452     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@maleadt maleadt merged commit 05959ad into master Dec 19, 2023
1 check passed
@maleadt maleadt deleted the tb/broadcast_unified branch December 19, 2023 15:00
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.

Broadcast does not preserve buffer type
1 participant