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.

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

Fix the dtInterval adjunstment on output time using adaptive and #1

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

JeroBnd
Copy link
Owner

@JeroBnd JeroBnd commented Sep 12, 2024

Problems with the precision of the time_to_output calculation cause errors in setting the correct time step at output time and final time in the em_b_wave ideal test (After removing the same mitigation code)

TYPE: bug fix

KEYWORDS: 5 to 10 words related to commit, separated by commas

SOURCE: Either "developer's name (affiliation)" .XOR. "internal" for a WRF Dev committee member

DESCRIPTION OF CHANGES:
Problem:
For the time_to_output calculation, the current simulation time is used as a float. This sometimes causes the time_to_output to be inaccurate, leading to problems, especially at the end of the simulation."

Solution:
I determinate the time_to_output using the current simulation time in TimeInterval type, avoiding the numeric.

ISSUE:
wrf-model#2103

LIST OF MODIFIED FILES:
/dyn_em/adapt_timestep_em.F

TESTS CONDUCTED:

RELEASE NOTE: Corrected source o problems realted to

Problems with the precision of the time_to_output calculation cause errors in setting the correct time step at output time and final time in the em_b_wave ideal test
@JeroBnd JeroBnd merged commit e590107 into master Sep 12, 2024
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.

1 participant