Skip to content

Update environmental data description in 13output.tex - #335

Open
e-perl-NOAA wants to merge 2 commits into
mainfrom
DD_output
Open

Update environmental data description in 13output.tex#335
e-perl-NOAA wants to merge 2 commits into
mainfrom
DD_output

Conversation

@e-perl-NOAA

Copy link
Copy Markdown
Collaborator

Include summary numbers as noted in ss3-source-code PR #780.

Include summary numbers as noted in ss3-source-code PR [#780](nmfs-ost/ss3-source-code#780 (review))
@github-actions

Copy link
Copy Markdown

Here are the artifacts from your PR:
textidote_report
SS330_User_Manual.pdf
Please review your changes in the linked artifacts.

@iantaylor-NOAA iantaylor-NOAA left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me. Thanks for the quick fix @e-perl-NOAA.

The order of the modified list is remains reversed from the description under "Specification of Time-Varying Parameters: Long Parameter Lines" but I think that's appropriate, because in that list, the bullets start at -1 and go to -4 whereas in this list it's describing column names in the order they appear (-4 to -1).

@e-perl-NOAA

Copy link
Copy Markdown
Collaborator Author

Thanks @iantaylor-NOAA, I changed it to echo that order and provided a link to that section

@github-actions

Copy link
Copy Markdown

Here are the artifacts from your PR:
textidote_report
SS330_User_Manual.pdf
Please review your changes in the linked artifacts.

@iantaylor-NOAA

Copy link
Copy Markdown
Contributor

@e-perl-NOAA, thanks, it is better like this and the link is great.

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.

2 participants