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.

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

Optionally make x,y,z,t available in ParsedAux #15877

Closed
dschwen opened this issue Sep 30, 2020 · 2 comments
Closed

Optionally make x,y,z,t available in ParsedAux #15877

dschwen opened this issue Sep 30, 2020 · 2 comments
Assignees
Labels
C: Framework 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 Sep 30, 2020

Reason

This has been requested before and is easy to implement. There are legitimate reasons to make current simulation time and QP coordinates available in ParsedAux for example to blend field variables as a function of time or space.

Design

Add boolean switch to make x,y,z,t available in the parsed expression. For backwards compatibility this defaults to false (in case field variables with the same names are already being used in existing input files)

Impact

Added capability.

@dschwen dschwen added T: task An enhancement to the software. P: normal A defect affecting operation with a low possibility of significantly affects. labels Sep 30, 2020
@dschwen dschwen self-assigned this Sep 30, 2020
dschwen added a commit to dschwen/moose that referenced this issue Sep 30, 2020
dschwen added a commit to dschwen/moose that referenced this issue Sep 30, 2020
dschwen added a commit to dschwen/moose that referenced this issue Oct 5, 2020
dschwen added a commit to dschwen/moose that referenced this issue Oct 5, 2020
dschwen added a commit to dschwen/moose that referenced this issue Oct 5, 2020
dschwen added a commit to dschwen/moose that referenced this issue Oct 5, 2020
dschwen added a commit to dschwen/moose that referenced this issue Oct 5, 2020
dschwen added a commit to dschwen/moose that referenced this issue Oct 6, 2020
@dschwen
Copy link
Member Author

dschwen commented Oct 13, 2020

Per @WilkAndy's comments I'm leaving this open for the next step, which is defaulting this option to true.

loganharbour pushed a commit to loganharbour/moose that referenced this issue Oct 29, 2020
loganharbour pushed a commit to loganharbour/moose that referenced this issue Oct 29, 2020
loganharbour pushed a commit to loganharbour/moose that referenced this issue Oct 29, 2020
jain651 pushed a commit to jain651/moose that referenced this issue Apr 19, 2021
jain651 pushed a commit to jain651/moose that referenced this issue Apr 19, 2021
jain651 pushed a commit to jain651/moose that referenced this issue Apr 19, 2021
@dschwen
Copy link
Member Author

dschwen commented Dec 31, 2023

Closed by #15900

@dschwen dschwen closed this as completed Dec 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: Framework P: normal A defect affecting operation with a low possibility of significantly affects. T: task An enhancement to the software.
Projects
None yet
Development

No branches or pull requests

2 participants