Skip to content

Default to no snapshots in output and supress a compile-time warning#653

Merged
einola merged 2 commits intodevelopfrom
minor_fixes
Jul 16, 2024
Merged

Default to no snapshots in output and supress a compile-time warning#653
einola merged 2 commits intodevelopfrom
minor_fixes

Conversation

@einola
Copy link
Copy Markdown
Member

@einola einola commented Jul 12, 2024

This PR consists of two commits:

Default to no snapshots in output

Set the default for snapshot output to zero, and make sure this actually
works!

... and ...

Suppress a compile-time warning

The sub-proc numbering function should throw an error if it doesn't find
what it's looking for.

einola added 2 commits July 12, 2024 09:00
Set the default for snapshot output to zero, and make sure this actually
works!
The sub-proc numbering function should throw an error if it doesn't find
what it's looking for.
@einola einola requested a review from tdcwilliams July 12, 2024 11:05
Copy link
Copy Markdown
Contributor

@tdcwilliams tdcwilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK. Made a suggestion that you can commit if you like

Comment thread model/finiteelement.cpp
@einola einola merged commit 0fee2e3 into develop Jul 16, 2024
@einola einola deleted the minor_fixes branch July 17, 2024 06:22
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

Successfully merging this pull request may close these issues.

2 participants