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

Add function to create parameter table #97

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

iantaylor-NOAA
Copy link
Contributor

This branch, created in March, should just be merged so the functions for parameter tables are more easily available to {sa4ss} users. I am interested in helping with additional tables functions as discussed in #78, but I don't see having time for that in the short term.

This PR adds adds functions for creating a table of parameters. The code is copied and adapted from lingcod.

Code like

cat("\\begin{landscape}")
table_pars_split(mod_base)
cat("\\end{landscape}")

will produces multiple tables with formatting like the snipped below. It's on the user to figure out how many tables get produced and add text like Parameters values are shown in Tables \ref{tab:table-pars-base-1}-\ref{tab:table-pars-base-16}. More detail (including info copied above) in: #78 (comment).

image

@iantaylor-NOAA iantaylor-NOAA marked this pull request as draft May 15, 2023 17:55
@iantaylor-NOAA
Copy link
Contributor Author

@kellijohnson-NOAA and I decided to postpone merging this pull request and instead will wait until after the 2023 PFMC assessments are complete in order to have more time to combine these changes with other table-related functions in a more comprehensive way.

I'm converting the PR to draft so the record remains.

For the 2023 petrale sole assessment, I will just add these functions (or something like them) to https://github.com/pfmc-assessments/petrale/.

@kellijohnson-NOAA kellijohnson-NOAA force-pushed the main branch 6 times, most recently from 85cf12c to dde22a3 Compare May 17, 2023 22:47
@kellijohnson-NOAA kellijohnson-NOAA force-pushed the main branch 2 times, most recently from 4a8d4ac to 865ba67 Compare July 11, 2023 19:41
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.

1 participant