-
Notifications
You must be signed in to change notification settings - Fork 31
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 MOPITT CO L3 pairing and updates to satplot #251
Conversation
Update personal branch to be in line with main develop branch
Removing a couple hacks made for an analysis.
Left in a personal copy change not pushing to larger group yet
Looks good to me and no clashes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These code changes look good to go to me. Happy for it to merged when ready.
@zmoon or @colin-harkins, this satellite comparison code looks ready to be merged into the develop branch to me. When you have time could you double check everything looks good? Thanks! - Rebecca |
Just checking in with the NOAA team on whether this can be merged into develop? |
Incorporation of MOPITT CO L3 pairing with averaging kernel application. Based on Rebecca Buchholz's implementation (https://github.com/NCAR/CAM-chem/blob/main/Python/advanced/apply_mopitt_AK_example.ipynb).
Updates to satplot: 1) swap order in spatial_bias_gridded to be model-obs, not obs-model; 2) include logic to take time mean of difference between l3 obs and model in spatial_bias_gridded.