-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Length based model 202204 #386
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
CHG: kLength::selectivities to use midpoints ADD: BH recruitment for length ADD: instantanesous mortality for length model
FIX: BH recruitment ADD: modelrunner test CHG: usermanual
ADD: Tagging length based process ADD: Category transition for length based UPD: user manual
Add unit test for Biomass-length based
…ING() is encountered
Users either choose to get Casal2 to sum proportions = 1 or keep them as is and Casal2 will flag a warning if they don't sum = 1
FIX: tag recapture and proportions mature obseravtions
FIX: reset call so growth gets updated
DEL: expand categories call should be done by config loader
Some model runners will get complex to get high precision objs across minimiser we willl require extensive estimation time. To avoid this set a lower tolerance and make sure minimisers get 'close' without taking a long time.
Some observation classes have different error tables or none at all.
These are depreciated and now handled by the r4Casal2 R package found here https://github.com/NIWAFisheriesModelling/Casal2_contrib/tree/main/r4Casal2
To replicate CASAL ratio qs penalty
Nuisance Q's need a prior. If a Q parameter is used in a ratio additional prior this caused an error. This accessor ignores additional prior types of ratio for the nuisance q class
CHG: disease mortality definition UPD: R package to read in selectivityby year report UPD: length based processes
ADD: compound selectivities
This needs to be done for age based methods as well. Sum to one needs to change the expected values.
ADD: growth verification MOVE: methods to be protected.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ADD: length based functionality for Casal2 with observations, processes and unit tests.
CHG Manual from one to and Age and Length manual.