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

Dsl/moist/radiation coupling - RadCouple Port in pyMoist #953

Merged
merged 12 commits into from
Jun 17, 2024

Conversation

mattweil11
Copy link

@mattweil11 mattweil11 commented Jun 13, 2024

Ticket [NDSL] Radiation Coupling #937
do_qa not implemented in radiation_coupling.py
Waiting on up to date testing data
Tests fail for CLDREFFL (5.55%) and CLDREFFI (5.08%)
Max Error set at 1e-9

@mattweil11 mattweil11 requested review from a team as code owners June 13, 2024 19:35
@FlorianDeconinck FlorianDeconinck added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Jun 13, 2024
@FlorianDeconinck
Copy link
Collaborator

0 diff explainer: this is related to the port of the Moist physics to the NDSL stack, it's execution and compilation is deactivated by default

Copy link
Collaborator

@FlorianDeconinck FlorianDeconinck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple of minor things, then we can merge

@FlorianDeconinck FlorianDeconinck merged commit 4317098 into dsl/develop Jun 17, 2024
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants