Skip to content

Conversation

@lindsayad
Copy link
Member

@lindsayad lindsayad commented Aug 4, 2025

We need to make sure that we properly size dof index containers to be inclusive of variables that are not defined on the current subdomain ID. So for a given element we can index into the container at that variable number and then obviously the associated vector of indices will be of zero length since there are no dof indices

Refs @GiudGiud's investigation on idaholab/moose#30357

We need to make sure that we properly size dof index containers
to be inclusive of variables that are not defined on the current
subdomain ID. So for a given element we can index into the container
at that variable number and then obviously the associated vector
of indices will be of zero length since there are no dof indices
@moosebuild
Copy link

Job Coverage, step Generate coverage on 2905cb5 wanted to post the following:

Coverage

ffdcb0 #4224 2905cb
Total Total +/- New
Rate 64.54% 64.54% -0.00% 50.00%
Hits 75837 75833 -4 1
Misses 41667 41671 +4 1

Diff coverage report

Full coverage report

Warnings

  • New new line coverage rate 50.00% is less than the suggested 90.0%

This comment will be updated on new commits.

@lindsayad lindsayad requested a review from roystgnr August 4, 2025 23:31
@roystgnr roystgnr merged commit a9f56e9 into libMesh:devel Aug 5, 2025
20 checks passed
@lindsayad lindsayad deleted the block-restriction-for-sc-dof-map branch August 5, 2025 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants