pfmc-assessments / sa4ss

Generate a stock assessment document from Stock Synthesis output
https://pfmc-assessments.github.io/sa4ss/
Other
12 stars 7 forks source link

Sub area models #91

Open chantelwetzel-noaa opened 1 year ago

chantelwetzel-noaa commented 1 year ago

Added new functionality to add identifier text to the reference label and tex file name for tables. This allows users to reference and include the same table from other model areas. As example, copper rockfish in CA is being modeled as two separate areas within the state. Adding "north" or "south" text to the beginning of a table label which allows each table to be referenced as unique tables in the text.

kellijohnson-NOAA commented 1 year ago

@chantelwetzel-noaa thanks for opening a pull request. I am just starting to look through this now and I am wondering if you can email me or provide a link to a pdf that uses this? I am struggling to see what the end out come is. Even a screen shot would work.

chantelwetzel-noaa commented 1 year ago

I don't have a fully flushed out example to share but here is a line where I test to make sure each table has it own unique numbering. I don't have the pdf built currently because I just broke it moving to a different model.

The two copper rockfish assessments this cycle will need to have a executive summary section that describes the results of both models (see Black Rockfish 2015 or China Rockfish 2017 document for example). Currently, the r4ss::SSexecutivesummary function creates csv files and with a fixed naming system and then the helper functions sa4ss:read_model and sa4ss::ex_table_tex creates tex files with the same names and lables like a_Catches_ES.tex and \ref{tab:removalsES}. This creates issues when needing to add tables from two separate models. This pull request adds the functionality for a user to add a unique name before each of the tex files and labels (north_a_Catches_ES.tex and \ref{tab:north-removalsES}).