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

AMD dependency information #22

Open
ivan-pi opened this issue May 27, 2022 · 4 comments
Open

AMD dependency information #22

ivan-pi opened this issue May 27, 2022 · 4 comments

Comments

@ivan-pi
Copy link

ivan-pi commented May 27, 2022

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:

#include "amd.h"

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).

@sfilippone
Copy link
Owner

Will have to update the docs before the next release.

@sfilippone
Copy link
Owner

BTW, the C version does not overwrite the input matrix (the Fortran version does according to the SuiteSparse docs).
In any case, AMD is an optional package

@sfilippone
Copy link
Owner

I added a small note in branch maint-3.8.0/README. Would you mind checking it is OK?
If it is satisfactory, I'll publish 3.8.0-2

@ivan-pi
Copy link
Author

ivan-pi commented Aug 5, 2022

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
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants