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

Rrfs v1.0.0 of the release with non-functional changes in modules and openmp directives #826

Draft
wants to merge 5 commits into
base: release/rrfs.v1.0.0
Choose a base branch
from

Conversation

TingLei-daprediction
Copy link
Contributor

This draft PR shows non-functional changes in compiling options and openmp directives to allow the PR #811 to pass the current ctests ( this PR also need similar non-functional changes, a corresponding draft PR would be submitted soon).

These draft PR are for evaluation/discussion of RRFS GSI developer to decide the efficient way to incorporate the optimization function in the above PR 811.

@RussTreadon-NOAA
Copy link
Contributor

@TingLei-daprediction , GSI issues require an issue. Which issue(s) are associated with the draft PR?

@TingLei-daprediction
Copy link
Contributor Author

TingLei-daprediction commented Jan 30, 2025 via email

@RussTreadon-NOAA
Copy link
Contributor

FYI: gitub CI fails for TingLei-daprediction:rrfs_v1.0.0_tl because this fork has diverged from NOAA-EMC:develop. The same statement applies to NOAA-EMC:release/rrfs.v1.0.0. PR #816 is not part the history for NOAA-EMC:release/rrfs.v1.0.0 or TingLei-daprediction:rrfs_v1.0.0_tl

@RussTreadon-NOAA RussTreadon-NOAA added the RRFS changes for RRS label Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RRFS changes for RRS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants