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

postprocess.cc needs at least one explicit time-dependent variable to work #210

Open
landinjm opened this issue May 7, 2024 · 2 comments

Comments

@landinjm
Copy link
Contributor

landinjm commented May 7, 2024

When trying to include post-processed fields for time independent variables you run into a divide-by-zero error. This likely has to do with the initialization (or lack thereof) of the invM used to calculated post-processed fields.

Including an auxiliary variable does not fix this issue.
Including an explicit variable does.

@landinjm
Copy link
Contributor Author

This is due to the post-processed fields piggy-backing on the main set of fields to use their DOFhandlers.

@landinjm landinjm changed the title postprocess.cc needs at least one explicit time-dependent variable to work postprocess.cc needs at least one explicit time-dependent variable to work Aug 19, 2024
@landinjm
Copy link
Contributor Author

landinjm commented Nov 5, 2024

Note that this also applies when the 0th field index is a vector field. Ideally, we should create the finite element spaces for post-processing separately. This would slightly increase memory overhead, but would fix this convoluted dependence of postprocess.cc and equations.cc. It wouldn't change the fact that post-processed fields are only calculated during outputs.

@landinjm landinjm self-assigned this Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant