Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Wrong sign for generalized Gumbel #13

Closed
jaburgoyne opened this issue Feb 4, 2022 · 1 comment
Closed

Wrong sign for generalized Gumbel #13

jaburgoyne opened this issue Feb 4, 2022 · 1 comment

Comments

@jaburgoyne
Copy link

I believe that the sign is incorrect in the parameterisation linking the generalized Gumbel distribution to its parent gamma-exponential distribution (8.4). Should it not be: GenGumbel(x; u, λ, n) = GammaExp(x; u + λ \ln n, λ, ν)? Version 1.0.0 of the field guide has a minus sign.
@gecrooks
Copy link
Owner

You're correct. Good catch.

gecrooks added a commit that referenced this issue Feb 15, 2022
@gecrooks
Fix parameterisation linking the generalized Gumbel distribution to i…
227ce09 
…ts parent gamma-exponential distribution (8.4). Resolve issue #13. Kudos:

John Ashley Burgoyne @jaburgoyne
gecrooks added a commit that referenced this issue Feb 15, 2022
@gecrooks
Fix parameterisation linking the generalized Gumbel distribution to i…
8f63e80 
…ts parent gamma-exponential distribution (8.4). Resolve issue #13. Kudos: John Ashley Burgoyne @jaburgoyne (#14)

* Fix parameterisation linking the generalized Gumbel distribution to its parent gamma-exponential distribution (8.4). Resolve issue #13. 
Kudos: John Ashley Burgoyne @jaburgoyne
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jaburgoyne @gecrooks