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

Doc tweaks #326

Merged
merged 10 commits into from Mar 24, 2017
Merged

Doc tweaks #326

merged 10 commits into from Mar 24, 2017

Conversation

jrleeman
Copy link
Contributor

Improvements on documentation including references and method documentation. Addresses #314

@dopplershift dopplershift modified the milestone: 0.5.0 Mar 10, 2017
@dopplershift dopplershift added Area: Docs Affects documentation Type: Enhancement Enhancement to existing functionality labels Mar 16, 2017
@jrleeman
Copy link
Contributor Author

Not the most elegant, but it does get the class methods documented. Thoughts @dopplershift?

@jrleeman jrleeman mentioned this pull request Mar 21, 2017
3 tasks
@jrleeman
Copy link
Contributor Author

I think we can safely ignore codecov here since it's griping about the references?

@dopplershift
Copy link
Member

Can you rebase and push? I'm concerned about the drop in coverage in the tests themselves.

Copy link
Member

@dopplershift dopplershift left a comment

Choose a reason for hiding this comment

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

Mostly just minor, adding the page numbers for the Hobbs 1977 references.

@@ -242,7 +224,7 @@ def heat_index(temperature, rh, mask_undefined=True):
def pressure_to_height_std(pressure):
r"""Convert pressure data to heights using the U.S. standard atmosphere.

The implementation uses the formula outlined in [7]_.
The implementation uses the formula outlined in [Hobbs1977]_.
Copy link
Member

Choose a reason for hiding this comment

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

Can you add the pages here?

@@ -273,7 +250,7 @@ def pressure_to_height_std(pressure):
def coriolis_parameter(latitude):
r"""Calculate the coriolis parameter at each point.

The implementation uses the formula outlined in [8]_.
The implementation uses the formula outlined in [Hobbs1977]_.
Copy link
Member

Choose a reason for hiding this comment

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

Here too.

@@ -339,15 +334,10 @@ def vapor_pressure(pressure, mixing):
Notes
-----
This function is a straightforward implementation of the equation given in many places,
such as [2]_:
such as [Hobbs1977]_:
Copy link
Member

Choose a reason for hiding this comment

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

Again.

@@ -478,15 +460,10 @@ def mixing_ratio(part_press, tot_press, molecular_weight_ratio=epsilon):
Notes
-----
This function is a straightforward implementation of the equation given in many places,
such as [5]_:
such as [Hobbs1977]_:
Copy link
Member

Choose a reason for hiding this comment

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

One more.

@@ -499,7 +476,7 @@ def saturation_mixing_ratio(tot_press, temperature):
r"""Calculate the saturation mixing ratio of water vapor.

This calculation is given total pressure and the temperature. The implementation
uses the formula outlined in [6]_.
uses the formula outlined in [Hobbs1977]_.
Copy link
Member

Choose a reason for hiding this comment

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

And here.

@@ -526,7 +498,7 @@ def equivalent_potential_temperature(pressure, temperature):
r"""Calculate equivalent potential temperature.

This calculation must be given an air parcel's pressure and temperature.
The implementation uses the formula outlined in [7]_.
The implementation uses the formula outlined in [Hobbs1977]_.
Copy link
Member

Choose a reason for hiding this comment

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

Again.

@dopplershift
Copy link
Member

One other issue is the colormap plot in the docs. This is what I see when I build locally:
plots-1

@jrleeman
Copy link
Contributor Author

Really? hmm. I rendered with RTD and didn't see that, but I'll change it up and see if I can reproduce.

@jrleeman
Copy link
Contributor Author

screenshot 2017-03-23 07 23 19

@dopplershift
Copy link
Member

Works for me if I add figsize=(6, 10) to the call to plt.subplots() in plots.rst.

@jrleeman
Copy link
Contributor Author

Modified the aspect ratio a bit - should be good to go.

@dopplershift dopplershift merged commit ec0d61f into Unidata:master Mar 24, 2017
@jrleeman jrleeman deleted the DocTweaks branch April 4, 2017 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Docs Affects documentation Type: Enhancement Enhancement to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants