Skip to content

JOSS review: Tolerance used in the speed comparison #78

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

Closed
ma-sadeghi opened this issue Jun 28, 2023 · 2 comments
Closed

JOSS review: Tolerance used in the speed comparison #78

ma-sadeghi opened this issue Jun 28, 2023 · 2 comments

Comments

@ma-sadeghi
Copy link

IIRC, TauFactor uses 5e-2 for flux tolerance. Have you made sure that you use an equivalent tolerance for the packages used in the speed comparison chart? I don't know about the rest, but I think PoreSpy uses 1e-8 for "residual" tolerance (might need some trial and error to see how that translates to flux tolerance).

@stke9
Copy link
Contributor

stke9 commented Jun 29, 2023

It's quite hard to calculate these equivalences, however, we decreased the tolerance on taufactor 2 until the converged value was closer to the 'true' value (calculated with a very low tolerance) than any of the other solvers. Let us know your thoughts on whether this is sufficient.

@ma-sadeghi
Copy link
Author

In that case, then your actual speedups should be slightly better (when done using an equivalent flux tolerance), which addresses my comment. Thanks!

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