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 MOPITT CO L3 pairing and updates to satplot #251

Merged
merged 9 commits into from
May 2, 2024

Conversation

mebruckner
Copy link
Collaborator

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.

mebruckner and others added 8 commits February 26, 2024 10:31
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
when pairing was performed for multiple model datasets scaling of mopitt observation occured each time. This carried over after pairing for each model because observation dataset was being shallow copied into the paired dataset.
@rrbuchholz
Copy link
Collaborator

Looks good to me and no clashes.

Copy link
Collaborator

@rrbuchholz rrbuchholz left a 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.

@rrbuchholz
Copy link
Collaborator

rrbuchholz commented Mar 13, 2024

@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

@rrbuchholz
Copy link
Collaborator

Just checking in with the NOAA team on whether this can be merged into develop?

@rrbuchholz rrbuchholz merged commit 06c5ff2 into NOAA-CSL:develop May 2, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants