Closed MOshima-PIFSC closed 2 weeks ago
Hello reviewer, thanks for taking the time to review this PR!
nit:
(for nitpicking) before making the suggestion. For example, nit:
I prefer using a data.frame()
instead of a matrix
because...This PR is now ready to be merged, no changes needed
.Part of #567
What is the feature?
How have you implemented the solution?
nyrs
is nown_yrs
nages
is nown_ages
weightatage
is nowweight_at_age
I wasn't sure if changing the variable names in test-fimsframe.R was necessary but I went ahead and changed them to be consistent.
nagecomp
is nown_age_comp
(in test-fimsframe.R)nyears
is nown_years
(in test-fimsframe.R)nindex
is nown_index
(in test-fimsframe.R)agecompdat
is nowage_comp_dat
(in test-fimsframe.R)fleet_names_agecomp
is nowfleet_names_age_comp
(in test-fimsframe.R)Does the PR impact any other area of the project?
Input
slot names are now in snake case
How to test this change
Changes were made in test-fimsframe.R as well.
Developer pre-PR checklist