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

fix regulatory calcualtions and add example on docs #285

Merged
merged 4 commits into from
Oct 7, 2024

Conversation

rschwant
Copy link
Collaborator

@rschwant rschwant commented Oct 5, 2024

  1. Fix MDA8 ozone calculations
  2. Add two days of output to ftp site for testing and ReadTheDocs example
  3. Add ReadTheDocs example for regulatory options
  4. Update a couple items to get rid of warnings (i.e. add numeric_only=True in two areas of driver)
  5. Add description for how read in code with the conda environment, so we are ready for this for next week
  6. Fix regulatory example in examples folder to exclude spatial overlay plot for consistency with ReadTheDocs
  7. Update docs in supported diagnostics to include 2 plots that do not work for regulatory options
  8. Updated version to 0.1 in prep for conda release

@rschwant rschwant requested review from zmoon and blychs October 5, 2024 00:02
Copy link
Collaborator

@blychs blychs left a comment

Choose a reason for hiding this comment

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

This looks great! We have to fix the Pandas issues, but wouldn't change it until we are sure.

docs/index.rst Outdated Show resolved Hide resolved
Co-authored-by: Zachary Moon <zmoon92@gmail.com>
Copy link
Collaborator

@blychs blychs left a comment

Choose a reason for hiding this comment

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

Great, thanks @rschwant!

@rschwant rschwant merged commit 84254a4 into NOAA-CSL:develop Oct 7, 2024
1 of 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
None yet
Development

Successfully merging this pull request may close these issues.

3 participants