Conversation
No space in report:47 between Mat_F_natage and sex_ratio as reported by @hgerritsen and @iagomosqueira
k-doering-NOAA
left a comment
There was a problem hiding this comment.
Thanks @kellijohnson-NOAA for fixing this! I confirmed using the Simple model, too that this change fixes the space issue.
Also, some accidental .sso files snuck into the commit; I pushed a commit to delete them so the only changes in the PR are to SS_write_report.tpl.
This is ready to merge in; I think I'll let @Rick-Methot-NOAA merge it in when ready.
|
ahh, I should of checked and not just blindly done |
|
This looks good. I'm deeply into another development branch right now, so will leave the cleanup and merging to Kelli and Kathryn |
What issue(s) does this PR address? Describe and add issue numbers, if applicable.
No space in report:47 between Mat_F_natage and sex_ratio
as reported by @hgerritsen and @iagomosqueira via email.
No issue was filed.
What tests have been done? Upload any model input files created for testing in a zip file, if possible.
Compiled the fix locally and reran the input files
provided by @hgerritsen.
Old output looked like
New output looks like
What tests/review still need to be done? Who can do it, and by when is it needed (ideally)?
Confirm that automated tests pass.
Check which is true. This PR requires:
Describe any changes in r4ss/SS3 manual/SSI/change log that are needed (if not checked):
I am uncertain if this would actually need to go in the change log?