-
Notifications
You must be signed in to change notification settings - Fork 1
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
Why do we add 1 here? #15
Comments
the parameter log_reck is estimated in log space. But the parameter reck must be > than 1.0, and cannot equal 1. Therefore, if you add 1, then even if log_reck is a negative number, this ensures that the constraint reck > 1 is met. |
Thanks, @smartell, your explanation makes perfect sense. An important point of clarification: if I'm using the estimated log_reck outside of the model to estimate brps, should I use the transformation? It obviously has a large impact on the results... AlaskaHerring/2019_forecast/r/brps.R Line 298 in 349a0de
|
Yes use the transformations
… On Aug 23, 2019, at 2:34 PM, Jane Sullivan ***@***.***> wrote:
Thanks, @smartell <https://github.com/smartell>, your explanatio makes perfect sense. An important point of clarification: if I'm using the estimated log_reck outside of the model to estimate brps, should I use the transformation? It obviously has a large impact on the results...
https://github.com/commfish/AlaskaHerring/blob/349a0deda1a557fd490358877f242e0ef309f557/2019_forecast/r/brps.R#L298 <https://github.com/commfish/AlaskaHerring/blob/349a0deda1a557fd490358877f242e0ef309f557/2019_forecast/r/brps.R#L298>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#15?email_source=notifications&email_token=AAJOESGRPU5JR2YFCDLYN73QGBJYDA5CNFSM4IPCJ6K2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5BMMHQ#issuecomment-524469790>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAJOESFFXVLDGGIK2QGD76LQGBJYDANCNFSM4IPCJ6KQ>.
|
AlaskaHerring/2019_forecast/admb/HER_best_condEffort.12_322/her.tpl
Line 1343 in 349a0de
The text was updated successfully, but these errors were encountered: