-
Notifications
You must be signed in to change notification settings - Fork 135
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
fix the stochastic collocation with gamma distribution #713
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the new gamma test analytic? Can we at least provide some guidance for the correct answers in case regolding is needed someday?
<description> | ||
This tests using SparseGridCollocation with gamma-distributed inputs with large scale parameters | ||
to test the fix for issue #693 | ||
</description> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this test not analytic?
@PaulTalbot-INL I created an analytic test for this PR. |
|
This is ready to merge except for the drafted email to the user group. |
Checklist passed... @wangcj05 Please email the users... Merging... |
Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
close #693
What are the significant changes in functionality due to this change request?
bug fix for #693
For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.
<internalParallel>
to True.