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

We should warn in the parsed phase field kernels if a free energy material property does not exist #6901

Closed
dschwen opened this issue May 4, 2016 · 0 comments
Closed

We should warn in the parsed phase field kernels if a free energy material property does not exist #6901

dschwen opened this issue May 4, 2016 · 0 comments
Assignees
@dschwen
Labels
C: Modules P: normal A defect affecting operation with a low possibility of significantly affects. T: task An enhancement to the software.

Comments

@dschwen
Copy link
Member

dschwen commented May 4, 2016

Description of the enhancement or error report

The CH/AC phase field kernels use only optional derivatives. So if the name of a free energy is misspelled all derivatives are non-existing and assumed to be 0.0. This should at least generate a warning. Ping @tonkmr

Rationale for the enhancement or information for reproducing the error

Helps the user find bugs in their inputs.

Identified impact

Improve user friendliness.
@dschwen dschwen self-assigned this May 4, 2016
@permcody permcody added T: task An enhancement to the software. P: normal A defect affecting operation with a low possibility of significantly affects. C: Modules labels May 4, 2016
dschwen added a commit to dschwen/moose that referenced this issue May 9, 2016
@dschwen
Add sanity check for free energy base names (idaholab#6901)
413126f
dschwen added a commit to dschwen/moose that referenced this issue May 10, 2016
@dschwen
Add sanity check for free energy base names (idaholab#6901)
17a54e9
dschwen added a commit to dschwen/moose that referenced this issue May 10, 2016
@dschwen
Add sanity check for free energy base names (idaholab#6901)
cc4dffc
@dschwen dschwen mentioned this issue May 10, 2016
Merged
dschwen added a commit to dschwen/moose that referenced this issue May 10, 2016
@dschwen
Add sanity check for free energy base names (idaholab#6901)
6aecdca
dschwen added a commit to dschwen/moose that referenced this issue May 10, 2016
@dschwen
Add sanity check for free energy base names (idaholab#6901)
e6bf9cd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dschwen dschwen

Labels
C: Modules P: normal A defect affecting operation with a low possibility of significantly affects. T: task An enhancement to the software.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dschwen @permcody