-
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
Add app info to exodus output #29786
Add app info to exodus output #29786
Conversation
we could just be consistent with the console and use this
from ConsoleUtils EDIT: now that I think about I dont want that. I m already on the fence for num processes though I dont have a strong opinion. |
Job Documentation, step Docs: sync website on 37650a7 wanted to post the following: View the site here This comment will be updated on new commits. |
Job Coverage, step Generate coverage on 37650a7 wanted to post the following: Framework coverage
Modules coverageCoverage did not change Full coverage reportsReports
This comment will be updated on new commits. |
b9fa0a5
to
6575156
Compare
Let’s squash these two commits together |
- n_processors, n_threads, and app name is now included in the input file included in the exodus output file closes idaholab#29767
6575156
to
37650a7
Compare
Thanks @DanielYankura and @GiudGiud ! |
Reason
Design
Impact
closes #29767