-
Notifications
You must be signed in to change notification settings - Fork 12
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
AMD dependency information #22
Comments
Will have to update the docs before the next release. |
BTW, the C version does not overwrite the input matrix (the Fortran version does according to the SuiteSparse docs). |
I added a small note in branch maint-3.8.0/README. Would you mind checking it is OK? |
Better than nothing. SuiteSparse and AMD are quite common so it should suffice. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The README doesn't specify in any detail how to obtain a copy of the AMD ordering routine. Adding a link to the SuiteSparse homepage, or the AMD User Guide from the SuiteSparse Github repository would be helpful.
I've noticed that only the C interface (i.e.
"amd.h"
) is actually needed:psblas3/util/psb_amd_order.c
Line 33 in b05679f
The AMD package provided in SuiteSparse however also includes two F77 routines, and a similar routine MC47 can be found in the Harwell Subroutine Library. I would recommend emphasizing that the C interface is used.
Otherwise one can get easily confused by the early versions of the AMD package which can be found in the
linalg/amd/
folder on Netlib, or directly from ACM TOMS Algorithm 837: AMD (Netlib mirror:toms/837
).The text was updated successfully, but these errors were encountered: