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

FEFaceEvaluation: Fix ECL for continuous elements #15527

Merged
merged 1 commit into from Jul 4, 2023

Conversation

bergbauer
Copy link
Contributor

@bergbauer bergbauer commented Jun 29, 2023

@peterrum @kronbichler

For special snowflakes like myself who need element-centric face integrals for continuous elements. ❄️

Copy link
Member

@kronbichler kronbichler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, apart from the initial setting. I have two questions, though:

  • Does the code now work as you expected?
  • Do you thin k we can extract a test case from your application? It would be great to find something to ensure that these complicated checks remain correct.

include/deal.II/matrix_free/fe_evaluation.h Outdated Show resolved Hide resolved
@kronbichler
Copy link
Member

/rebuild

@bergbauer
Copy link
Contributor Author

Does the code now work as you expected?

Yes, I get the same results for ECL and FCL.

Do you think we can extract a test case from your application? It would be great to find something to ensure that these complicated checks remain correct.

I am thinking of a check that compares results from ECL with FCL for FE_Q for a normal gradient jump term.

@bergbauer
Copy link
Contributor Author

I had problems with MatrixFreeTools::categorize_by_boundary_ids(tria, additional_data);. Do you have an idea why this does not work? @peterrum

The test uses a mask instead of categorization by boundary ids.

@peterrum
Copy link
Member

I had problems with MatrixFreeTools::categorize_by_boundary_ids(tria, additional_data);. Do you have an idea why this does not work?

What is the problem?

@bergbauer
Copy link
Contributor Author

bergbauer commented Jun 30, 2023

What is the problem?

The computed dof_indices are wrong.

dof_indices[v] =
dof_info.dof_indices.data() + my_index_start[0].first;

tests/matrix_free/ecl_05.cc Show resolved Hide resolved
@tamiko tamiko added this to the Release 9.6 milestone Jul 1, 2023
@kronbichler
Copy link
Member

Thank you for the test.

@bergbauer
Copy link
Contributor Author

In reference to #15589 use EvaluationFlags interface 6681f2b

@kronbichler
Copy link
Member

Please squash to a smaller number of commits, then this is ready.

@bergbauer
Copy link
Contributor Author

Squashed! @kronbichler

@tamiko tamiko merged commit 2050f26 into dealii:master Jul 4, 2023
11 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants