Skip to content

[WIP] Feature/magma free#1

Draft
a-sevin wants to merge 4 commits intodevfrom
feature/magma-free
Draft

[WIP] Feature/magma free#1
a-sevin wants to merge 4 commits intodevfrom
feature/magma-free

Conversation

@a-sevin
Copy link
Member

@a-sevin a-sevin commented Sep 27, 2020

This is a proposition to remove magma dependency (but still optional)

When I writing this PR, it's not compiling on ArchLinux but it does on CeontOS 8 (could be a bug of the arch gcc).
What's changing ?

  • cudacomppsinv uses CUDACOMP_magma_compute_SVDpseudoInverse_cli with USE_MAGMA=ON else CUDACOMP_cusolver_compute_SVDpseudoInverse_cli
  • there are 2 new CLI functions :
    • cudacomppsinv_magma for CUDACOMP_magma_compute_SVDpseudoInverse_cli
    • cudacomppsinv_cusolver for CUDACOMP_cusolver_compute_SVDpseudoInverse_cli

I also reworked cudacomp_types.h to be remove MAGMA functions calls.

@DasVinch
Copy link
Member

Thanks Arnaud. Note that if the speed is maintained (RT MVM: critical; SVDs: non critical but convenient), we have no problems with dropping magma altogether, to get rid of some #ifdefs.

@a-sevin a-sevin marked this pull request as draft September 28, 2020 17:37
@a-sevin a-sevin force-pushed the feature/magma-free branch from 43ad960 to d3eaeb8 Compare April 12, 2021 08:29
@gregwallan
Copy link

What is the status of removing the MAGMA dependency?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants