-
Notifications
You must be signed in to change notification settings - Fork 133
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
Failure with GA #1677
Failure with GA #1677
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.
Changes are good to me. One question: is it possible to add a test? @aalfonsi
@mandd @Jimmy-INL Do you have any comments on this PR?
@wangcj05 Yes, if it is possible, I would like Diego (@mandd) and Mohammad (@Jimmy-INL) to test this with imposed failures (of the model) to see if the behavior of the optimizer is as expected (I am not super sure). |
…b.com/idaholab/raven into Jimmy-testingAndreasfixForFailingCodesinGA
Are we good with this? The Mac test machine does not look like to work. I tested on my MAC and everything PASS:
|
Job Test mac on b0bcbeb : invalidated by @Jimmy-INL |
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.
I am OK with this PR, @aalfonsi please generate an issue to address the random failures.
since everything is approved, is now mergeable? (Can anybody merge it? :D ) |
@Jimmy-INL Can you approve and merge it? |
I am OK, but please @aalfonsi generate an issue about random failures. |
@wangcj05: are you ok if we merge this PR? |
@mandd Yes, we can merge it. |
All reviewers agree the changes, and This PR can be merged.
checklist is good, and PR can be merged. |
@aalfonsi I have merged your PR, thanks for your contribution. @Jimmy-INL I'm sorry I need to dismiss your review in order to merge this PR. |
Thank you all! |
Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Closes #1676
What are the significant changes in functionality due to this change request?
Allow the GA to handle the failure
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.raven/tests/framework/user_guide
andraven/docs/workshop
) have been changed, the associated documentation must be reviewed and assured the text matches the example.