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
Check to see if a function name could evaluate to a parsed function #8412
Conversation
If people do that it is retarded. Just delete those function blocks and the tests/inputs should still pass/run. |
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.
Looks good otherwise
@@ -29,5 +29,9 @@ AddFunctionAction::AddFunctionAction(InputParameters params) : | |||
void | |||
AddFunctionAction::act() | |||
{ | |||
FunctionParserBase<Real> fp; | |||
std::string vars = "x,y,z,t,NaN,pi,e"; |
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.
we may want to make this a static constant member
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.
Ah, wait, this only occurs in libmesh. Well, nevermind then.
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.
Nope, the same string appears in FEProblemBase::getFunction
. No big deal, but this was changed before (when NaN was added by Roy).
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.
Yes, that is where I grabbed it. It would be nice if this sort of thing lived in a common place.
60e8eab
to
3569c25
Compare
Job |
1 similar comment
Job |
3569c25
to
f1c5891
Compare
This just gives a warning, wasn't sure if it should be an error.
This also ignores
ParsedFunctions
. There seems to be several places where peoplehave something like:
I wasn't sure if you wanted to warn/error there as well.
closes #7659