-
Notifications
You must be signed in to change notification settings - Fork 33
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
Update docs: Fix links, Add aircraft plots, Update yaml descriptions, Update observation description #342
base: develop
Are you sure you want to change the base?
Conversation
I just added a note for the CONUS extent to this PR too. This will close this PR too: domain_type == 'all' behavior I also created a new issue and put it in version 2 to generalize this extent feature: #343. Generalizing this now since it is used in several examples seems to rushed. Better to document current capabilities and we can advance this further in version 2. |
Co-authored-by: Zachary Moon <[email protected]>
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.
A few initial comments
..._static/figures/plot_grp2.vertprofile.NO2_LIF.2023-06-27_00.2023-06-28_23.all.LosAngeles.png
Outdated
Show resolved
Hide resolved
This is ready for review when you all have time. |
This updates the docs and will close the following issues:
#224
#205
#293
#294
#295
During this process, I realized that some of the YAML options in the aircraft plots are not fully connected but still listed in the example YAML files and or need some cleaning up. I created a new issue for this that we will address in Version 2. All YAML options described on the ReadTheDocs YAML appendix description seem to be working as specified and represent the base options available so far for these plots. Clean up aircraft plotting example YAML files #341.