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

Add variable rescaling for aerosol species #1339

Merged
merged 18 commits into from
Nov 20, 2024

Conversation

andytangborn
Copy link
Collaborator

@andytangborn andytangborn commented Oct 18, 2024

This is a draft PR for additions to chem_diagb.h that could allow for a variable rescaling (rs) for each aerosol species. It does not yet include a way to read the values in from a yaml or config file. The it compiles and runs, but I have not yet tested it using 3dvar. And it still has lots of print statements used for testing. It runs in about 40 seconds at C96, and is probably slowed down by the print statements. Any comments will be helpful.

Also, note the if statement. I was unsure of the dimensions of an array, so I put in a test. I'll remove it along with the print statements.

… aerosol species. Does not yet include a way to read the values in from a yaml or config file.
@andytangborn andytangborn marked this pull request as ready for review November 14, 2024 20:20
Copy link
Contributor

@CoryMartin-NOAA CoryMartin-NOAA left a comment

Choose a reason for hiding this comment

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

One minor cleanup thing and then I think this is ready to go

utils/chem/chem_diagb.h Outdated Show resolved Hide resolved
@CoryMartin-NOAA CoryMartin-NOAA merged commit 15c29b1 into develop Nov 20, 2024
5 checks passed
@CoryMartin-NOAA CoryMartin-NOAA deleted the feature/variable_aero_rs branch November 20, 2024 18:25
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.

2 participants