Skip to content

Update pycmap_cruise_ST_bounds.rst #59

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.

Sign up for GitHub

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

Jump to bottom
Open
MSchatzUW wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update pycmap_cruise_ST_bounds.rst #59

MSchatzUW wants to merge 2 commits into from

Conversation

@MSchatzUW
Copy link
Contributor

@MSchatzUW MSchatzUW commented May 15, 2020

Megan's comments

MSchatzUW
MSchatzUW commented May 28, 2020
View reviewed changes
Copy link
Contributor Author

@MSchatzUW MSchatzUW left a comment

Choose a reason for hiding this comment

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

Removed Comments below:

"Returns a dataframe containing the spatio-temporal bounding box accosiated with the specified cruise. Effectively, this method returns a subset of the outputs returned by the cruise_by_name method."

..COMMENT: "cruise_by_name" method should be "Cruise Detail By Name" method so that it is consistent with the title name of the page. And you probably want to make this a hyperlink.

" :Parameters:
cruiseName: string
The official cruise name. If applicable, you may also use cruise "nickname" ('Diel', 'Gradients_1' ...).
A full list of cruise names can be retrieved using cruise method."

..COMMENT: Again, you probably want "cruise method" above to be a hyperlink to the "Cruise Detail By Name" method.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MSchatzUW