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

Lopez lightning revisions #284

Merged
merged 2 commits into from
Oct 30, 2024
Merged

Conversation

mmanyin
Copy link
Contributor

@mmanyin mmanyin commented Oct 29, 2024

Modified Lopez lightning flash rate to use a different approach for convective-cloud-base. Tuned Lopez scheme for c90 and c180 Free-run and Replay. Made LOPEZ the default lightning scheme.
This PR is zero-diff for PCHEM simulations, non-zero-diff for GMI since lightning is a source of NOx.

@mmanyin mmanyin added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Oct 29, 2024
@mmanyin mmanyin requested review from a team as code owners October 29, 2024 22:47
@mathomp4
Copy link
Member

Huh. Weird CI error. I've pushed a fix that should let the YAML validator work.

@mmanyin mmanyin merged commit ef6e5e1 into develop Oct 30, 2024
11 checks passed
@mmanyin mmanyin deleted the feature/mmanyin/revised_lopez_lightning branch October 30, 2024 14:21
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.

2 participants