-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Adds postprocessor to track time step number for transient problems #29128
base: next
Are you sure you want to change the base?
Conversation
Minor quality-of-life improvement for users who want to track time step counts during their MOOSE simulations. closes idaholab#29127
b584fee
to
c915948
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice this is almost ready
let's squash all these commits togther in just a
commit message |
6289ad6
to
6066950
Compare
Job Documentation, step Docs: sync website on ac0ce11 wanted to post the following: View the site here This comment will be updated on new commits. |
6066950
to
c3b81ba
Compare
c3b81ba
to
3c176f1
Compare
Job Coverage, step Generate coverage on 20aa5b1 wanted to post the following: Framework coverage
Modules coverageCoverage did not change Full coverage reportsReports
This comment will be updated on new commits. |
- updated wording, file names based on suggestions - added docstrings - simplified test, changed dt
3c176f1
to
20aa5b1
Compare
Apply suggestions from code review corrects code listing path
ac0ce11
to
36121e8
Compare
Minor quality-of-life improvement for users who want to track time step counts during their MOOSE simulations.
closes #29127