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

sync_times_object does not force solution to occur #25368

Closed
joshuahansel opened this issue Aug 31, 2023 · 0 comments · Fixed by #25371
Closed

sync_times_object does not force solution to occur #25368

joshuahansel opened this issue Aug 31, 2023 · 0 comments · Fixed by #25371
Labels
C: Framework P: normal A defect affecting operation with a low possibility of significantly affects. T: defect An anomaly, which is anything that deviates from expectations.

Comments

@joshuahansel
Copy link
Contributor

Bug Description

Using the sync_times_object in an output object does not actually force solution to occur at the times, just outputs the solution if the time happens to be in the list (when using sync_only = true). If sync_only = false, the parameter appears to have no effect.

Steps to Reproduce

Run the following input file and look at the CSV output times:
sync_times.txt

Impact

This prevents users from getting much use from this option.
@joshuahansel joshuahansel added T: defect An anomaly, which is anything that deviates from expectations. P: normal A defect affecting operation with a low possibility of significantly affects. C: Framework labels Aug 31, 2023
joshuahansel added a commit to joshuahansel/moose that referenced this issue Aug 31, 2023
@joshuahansel
Fixed sync_times_object in Output
4f7a1ed 
Closes idaholab#25368
@joshuahansel joshuahansel mentioned this issue Aug 31, 2023
Merged
joshuahansel added a commit to joshuahansel/moose that referenced this issue Aug 31, 2023
@joshuahansel
Added newsletter entry for sync_times_object fix
5252c6f 
Refs idaholab#25368
joshuahansel added a commit to joshuahansel/moose that referenced this issue Aug 31, 2023
@joshuahansel
Updated expect_err in sync times test
6d6e693 
Refs idaholab#25368
joshuahansel added a commit to joshuahansel/moose that referenced this issue Sep 1, 2023
@joshuahansel
Fixed sync_times_object in Output
e04d483 
Closes idaholab#25368
oanaoana pushed a commit to oanaoana/moose that referenced this issue Oct 19, 2023
@joshuahansel @oanaoana
Fixed sync_times_object in Output
c9e6620 
Closes idaholab#25368
oanaoana pushed a commit to oanaoana/moose that referenced this issue Nov 7, 2023
@joshuahansel @oanaoana
Fixed sync_times_object in Output
8a5f978 
Closes idaholab#25368
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C: Framework P: normal A defect affecting operation with a low possibility of significantly affects. T: defect An anomaly, which is anything that deviates from expectations.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed sync_times_object in Output joshuahansel/moose
1 participant
@joshuahansel