-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Added error for when no parameter fits #29734
base: next
Are you sure you want to change the base?
Added error for when no parameter fits #29734
Conversation
- added check to applyParameter that throws an error if the provided param does not match any known param - added to applyParameter unit test to ensure an error is thrown closes idaholab#27556
Job Documentation, step Docs: sync website on a66e8c0 wanted to post the following: View the site here This comment will be updated on new commits. |
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'm looking around to see if we can provide any more additional context to the error
@@ -16,6 +16,7 @@ | |||
#include "MultiMooseEnum.h" | |||
#include "ExecFlagEnum.h" | |||
#include "MooseObject.h" | |||
#include "FEProblem.h" |
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.
What is this include needed for?
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.
That is odd, I don't remember adding that? Doesn't seem like I need it though so I'll go ahead and remove it.
Job Coverage, step Generate coverage on a66e8c0 wanted to post the following: Framework coverage
Modules coverageCoverage did not change Full coverage reportsReports
This comment will be updated on new commits. |
Reason
Design
Impact
New error check
closes #27556