- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1
adding amdgpu ext skeleton. #88
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
Conversation
| Codecov ReportAttention: Patch coverage is  
 
 Additional details and impacted files@@            Coverage Diff             @@
##             main      #88      +/-   ##
==========================================
- Coverage   61.81%   60.95%   -0.87%     
==========================================
  Files          35       36       +1     
  Lines        1616     1639      +23     
==========================================
  Hits          999      999              
- Misses        617      640      +23     ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
 | 
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.
The syntax looks ok to me and it compiled on my RX 7600 GPU. Now I need to figure out how to test it.
…d error currently in the cg solve. Not too familliar with HIP so it might take a minute to figure this out.
| 
 I added an example under  This was tested on a RX 7900 I picked up yesterday. Maybe I still have some driver kinks to work out.  | 
…in julia can't handle a minus sign or being multiplied by a constant. Need to file an issue over there or a PR
| @lxmota the AMDGPU v1.3 release seems to have fixed the issues I had on my machine. Looks like we're up and running for both CUDA and AMD. Metal isn't even worth talking about until Apple decides to support double precision. | 
Accidentally deleted the old PR.