[Bug]: SSexecutivesummary(tables = ())
only works for a subset of combinations
#888
Open
1 task done
Labels
bug
low priority
A low-priority item that does NOT have to be worked on
output
issues related to output processing
Projects
Describe the bug
In 86f4b21 @iantaylor-NOAA added the capacity to produce a selection of the tables output by
SSexecutivesummary()
by adding the tables argument but some tables depend on information created from previous tables. In an offline conversation we discussed how users could build all tables, move them to a new location, and augment them via a script rather than having to worry about rerunningSSexecutivesummary()
and having it mess up tables that they had augmented by hand. The function only takes ~15 seconds to run and therefore being able to create individual tables is not a high priority. I suggest that we remove this feature that allows some tables to be made by themselves but not all. This will increase the readability of the code by decreasing the number of if statements. Maybe even speed it up a little bit for the case when every table is created by default. If we choose to leave the feature in, then we really should augment the code to allow users to run each table individually.To Reproduce
Expected behavior
The documentation for the function does warn users that not all tables are available as individual tables but the function call does not indicate which should NOT be removed from the input vector, which is conditional on what you want. If a table cannot be produced alone and we leave the code we should at least give warnings to the user rather than the warnings listed above.
Screenshots
No response
Which OS are you seeing the problem on?
Windows
Which versions of r4ss and SS3 are you seeing the problem on?
Since v1.46.1
Additional Context
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: