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

WIP: continuing higher-level wrappers #32

Open
wants to merge 133 commits into
base: master
Choose a base branch
from

Conversation

JerryChen97
Copy link
Collaborator

Just for polishing, e.g. fixing the getrs bug, changing into more compact forms, adding more detailed comments and e.t.c.

@JerryChen97
Copy link
Collaborator Author

I think now all the higher-level wrappers for linear system solvers have been finished, including gesv, gerbt, gels, posv, hesv and sysv.
If available, @Roger-luo please help review the current codes and I will create another branch and continue on the factorization subroutines and eigenvalue subroutines. (Also SVD subroutines have been completed before, including gesvd, gesdd and gebrd.)

@JerryChen97 JerryChen97 requested a review from Roger-luo August 28, 2019 19:30
@JerryChen97 JerryChen97 self-assigned this Aug 28, 2019
@JerryChen97
Copy link
Collaborator Author

How about I finish the left higher wrappers for eigenvalues? Or just first consider QR LU & svd dispatching and come back to these stuff after anything else which is more important(e.g. dispatch & BB) is set up?@Roger-luo

return type == Float64 || type == ComplexF64 ? Float64 : Float32
end

for type in magmaTypeList_real
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you don't need to create the symbols, just use type.

macro magmafunc_generic(function_name, str)
return Expr(:quote, Symbol(function_name, str))
end

const magmaTypeTuple= (Float32, Float64, ComplexF32, ComplexF64)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is sufficient. you don't need to parse strings manually.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is sufficient. you don't need to parse strings manually.

Ah cool. There is some improvisation made by me a long time ago for convenience. Thank you for your careful checking and improvement.

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.

2 participants