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.

Sign up for GitHub

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

Jump to bottom

Unify number of significant digit in Log terminal #1096

Closed
blaisb opened this issue Apr 18, 2024 · 0 comments · Fixed by #1310
Closed

Unify number of significant digit in Log terminal #1096

blaisb opened this issue Apr 18, 2024 · 0 comments · Fixed by #1310
Assignees
@AmishgaAlphonius

Comments

@blaisb
Copy link
Contributor

blaisb commented Apr 18, 2024

As noted by @PierreLaurentinCS , the number of significant digits and the notation (fixed vs scientific) in the post-processing capabilities is not uniform. This should be uniformized in the future.
@AmishgaAlphonius AmishgaAlphonius self-assigned this Oct 3, 2024
@AmishgaAlphonius AmishgaAlphonius mentioned this issue Oct 4, 2024
Merged
12 tasks
@AmishgaAlphonius AmishgaAlphonius linked a pull request Oct 4, 2024 that will close this issue
Uniformize postprocess output precision #1310
Merged
12 tasks
blaisb added a commit that referenced this issue Oct 6, 2024
@AmishgaAlphonius @blaisb
Uniformize postprocess output precision (#1310)
ecb85df 
Description
In response to the the issue #1096, this PR uniformize precision of outputted values using the log precision parameter. All postprocessing tables outputted on terminal are now displayed in scientific notation.

Testing
All affected tests were updated.

Co-authored-by: Bruno Blais <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AmishgaAlphonius AmishgaAlphonius

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Uniformize postprocess output precision chaos-polymtl/lethe
2 participants
@blaisb @AmishgaAlphonius