Skip to content

Update par_tables_fn.R - MV Tweedie Reporting#92

Open
emilyliljestrand wants to merge 2 commits into
timjmiller:masterfrom
emilyliljestrand:MVTweedieFix
Open

Update par_tables_fn.R - MV Tweedie Reporting#92
emilyliljestrand wants to merge 2 commits into
timjmiller:masterfrom
emilyliljestrand:MVTweedieFix

Conversation

@emilyliljestrand

Copy link
Copy Markdown

I proposed two changes to how MVTweedie parameters are being reported in the report table. I changed "power" to psi, which is the notation from the Thorson paper, and I corrected how the value is being reported (needed to add 1, so its bounded between 1 and 2).
https://academic.oup.com/icesjms/article/80/10/2630/6710216

changed labeling on mvtweedie and fixed how it was being reported in the output table
Comment thread R/par_tables_fn.R

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Let's use the "p" label for defining the parameter. This seems pretty common in the literature on the Tweedie and is also used by the TMB function.

Changed psi to p in the output for the multivariate tweedie distribution
@emilyliljestrand

Copy link
Copy Markdown
Author

cool. Just changed it back - psi to p

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.

2 participants