-
Notifications
You must be signed in to change notification settings - Fork 15
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
Factorizing matrices with pypardiso is not a no-op #98
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Factorizing matrices with pypardiso is not a no-op and more importantly, can have substantial benefit to calculation speed.
While I don't mind warning users of potential limited benefit, I think it's not wise to outright deny users to factorize their
A
matrices through the below code.brightway2-calc/bw2calc/lca_base.py
Lines 119 to 120 in adc379e
See also haasad/PyPardiso#75 for further discussion.
edit: Also see the
Pardiso
documentation on solver phases,A
matrix decomposition is possiblehttps://www.intel.com/content/www/us/en/docs/onemkl/developer-reference-c/2024-1/pardiso.html#GUID-FB4D5442-2BE8-4687-A6F6-D2AAE34B8A87
The text was updated successfully, but these errors were encountered: