Skip to content

Conversation

@Rick-Methot-NOAA
Copy link
Collaborator

@Rick-Methot-NOAA Rick-Methot-NOAA commented Nov 26, 2025

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

Move calculation of HCR_anchor to allow correct function in mceval

What tests have been done?

Where are the relevant files?

<-- - [x] Test files are in the issue. -->

What tests/review still need to be done?

none

Is there an input change for users to Stock Synthesis?

Additional information (optional).

Copy link
Contributor

@iantaylor-NOAA iantaylor-NOAA left a comment

Choose a reason for hiding this comment

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

I don't understand why this fixed worked, but it's a simple change to remove the echoinput with zero potential side effects problems, and seems to have fixed the issue for me.

@Rick-Methot-NOAA
Copy link
Collaborator Author

The fix was to move the code for calc of HCR_anchor out of the conditional if(show_MSY==1).

@iantaylor-NOAA
Copy link
Contributor

@Rick-Methot-NOAA, is there an additional change that didn't get committed?
Under "Files changed" https://github.com/nmfs-ost/ss3-source-code/pull/744/files I'm only seeing removal of the echoinput line:
image

@Rick-Methot-NOAA
Copy link
Collaborator Author

I messed up my github action so accidentally committed the real change to main first.

@iantaylor-NOAA
Copy link
Contributor

Aha, no wonder the fix worked. No problem.

@Rick-Methot-NOAA Rick-Methot-NOAA merged commit 759ea31 into main Dec 1, 2025
21 checks passed
@Rick-Methot-NOAA Rick-Methot-NOAA deleted the move_hcr_anchor branch December 1, 2025 18:10
@Rick-Methot-NOAA
Copy link
Collaborator Author

@e-perl-NOAA Please check to see if the history is OK.

@e-perl-NOAA
Copy link
Collaborator

I've squashed the two commits so the history is good to go now.

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]: missing values for forecast derived quantities in MCMC

4 participants