Skip to content

Comments

increase have_data storage to 150.#624

Merged
Rick-Methot-NOAA merged 1 commit intomainfrom
622-bug-unable-to-include-multiple-caal-records-for-same-fleet-year-and-bin-with-different-ageing-errors
Sep 25, 2024
Merged

increase have_data storage to 150.#624
Rick-Methot-NOAA merged 1 commit intomainfrom
622-bug-unable-to-include-multiple-caal-records-for-same-fleet-year-and-bin-with-different-ageing-errors

Conversation

@Rick-Methot-NOAA
Copy link
Collaborator

Concisely describe what has been changed/addressed in the pull request.

simple increase in storage of have_data array to allow more CAAL observations. New limit is 150.

What tests have been done?

revised sablefish model runs with this increased storage

Where are the relevant files?

What tests/review still need to be done?

none

Is there an input change for users to Stock Synthesis?

[x] No, there was no input change. -->

If there is no issue related to this PR, replace this text with your example stock synthesis input.

-->

Additional information (optional).

Copy link

@chantelwetzel-noaa chantelwetzel-noaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for fixing this so quickly. Everything looks good to me.

@Rick-Methot-NOAA Rick-Methot-NOAA merged commit 7adcf1b into main Sep 25, 2024
@Rick-Methot-NOAA Rick-Methot-NOAA deleted the 622-bug-unable-to-include-multiple-caal-records-for-same-fleet-year-and-bin-with-different-ageing-errors branch September 25, 2024 18:45
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.

[Bug]: Unable to include multiple CAAL records for same fleet, year, and bin with different ageing errors

2 participants