Skip to content

Fixed the boundary check in PatchSidesetGenerator#33187

Merged
joshuahansel merged 3 commits into
idaholab:nextfrom
joshuahansel:graydiffuse-nonexistent-boundary
Jul 1, 2026
Merged

Fixed the boundary check in PatchSidesetGenerator#33187
joshuahansel merged 3 commits into
idaholab:nextfrom
joshuahansel:graydiffuse-nonexistent-boundary

Conversation

@joshuahansel

Copy link
Copy Markdown
Contributor

The boundary-exists check in PatchSidesetGenerator was broken due to using the wrong ID type, which didn't respect the invalid ID check.

Closes #32724

@joshuahansel joshuahansel self-assigned this Jun 24, 2026
@joshuahansel joshuahansel requested a review from lindsayad as a code owner June 24, 2026 16:14
@moosebuild

moosebuild commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Job Documentation, step Docs: sync website on 5aa0b26 wanted to post the following:

View the site here

This comment will be updated on new commits.

@joshuahansel joshuahansel marked this pull request as draft June 24, 2026 18:49
@moosebuild

moosebuild commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Job Coverage, step Generate coverage on 5aa0b26 wanted to post the following:

Framework coverage

d7c4bd #33187 5aa0b2
Total Total +/- New
Rate 85.99% 85.99% -0.00% 100.00%
Hits 136168 136168 - 20
Misses 22185 22189 +4 0

Diff coverage report

Full coverage report

Modules coverage

Heat transfer

d7c4bd #33187 5aa0b2
Total Total +/- New
Rate 89.10% 89.11% +0.01% 100.00%
Hits 4758 4756 -2 4
Misses 582 581 -1 0

Diff coverage report

Full coverage report

Reactor

d7c4bd #33187 5aa0b2
Total Total +/- New
Rate 94.03% 94.03% - 100.00%
Hits 8428 8428 - 1
Misses 535 535 - 0

Diff coverage report

Full coverage report

Full coverage reports

Reports

This comment will be updated on new commits.

@joshuahansel

Copy link
Copy Markdown
Contributor Author

This will proceed after #33190

@joshuahansel joshuahansel force-pushed the graydiffuse-nonexistent-boundary branch from a5163e6 to 85aafba Compare June 25, 2026 12:23
@joshuahansel

Copy link
Copy Markdown
Contributor Author

This depends on #33190 - I'll rebase and mark ready when that one is merged.

@joshuahansel joshuahansel force-pushed the graydiffuse-nonexistent-boundary branch from 8985d1c to 6eaeaa8 Compare June 29, 2026 12:49
@joshuahansel joshuahansel force-pushed the graydiffuse-nonexistent-boundary branch from 6eaeaa8 to 5aa0b26 Compare June 30, 2026 15:12
@moosebuild

Copy link
Copy Markdown
Contributor

Job Test, step Results summary on 5aa0b26 wanted to post the following:

Framework test summary

Compared against d7c4bde in job civet.inl.gov/job/3938647.

No change

Modules test summary

Compared against d7c4bde in job civet.inl.gov/job/3938647.

Removed tests

Test Time (s) Memory (MB)
heat_transfer/test:generate_radiation_patch.generate_radiation_patch_linear 0.66 105.19
heat_transfer/test:generate_radiation_patch.generate_radiation_patch 0.65 111.52
heat_transfer/test:generate_radiation_patch.generate_radiation_patch_grid 0.64 109.35
heat_transfer/test:generate_radiation_patch.generate_radiation_patch_centroid 0.62 114.28
heat_transfer/test:generate_radiation_patch.generate_radiation_patch_centroid_error 0.54 27.71
heat_transfer/test:generate_radiation_patch.generate_radiation_patch_grid_2D 0.53 101.05
heat_transfer/test:generate_radiation_patch.generate_radiation_patch_grid_2D_overpart 0.50 87.14

Added tests

Test Time (s) Memory (MB)
heat_transfer/test:meshgenerators/patch_sideset_generator.patch_sideset_generator 0.70 80.37
heat_transfer/test:meshgenerators/patch_sideset_generator.patch_sideset_generator_grid 0.69 83.28
heat_transfer/test:meshgenerators/patch_sideset_generator.patch_sideset_generator_linear 0.69 105.99
heat_transfer/test:meshgenerators/patch_sideset_generator.patch_sideset_generator_centroid 0.66 105.36
heat_transfer/test:meshgenerators/patch_sideset_generator.patch_sideset_generator_centroid_error 0.59 83.24
heat_transfer/test:meshgenerators/patch_sideset_generator.patch_sideset_generator_grid_2D_overpart 0.57 65.55
heat_transfer/test:radiation_transfer_action.nonexistent_boundary 0.54 91.15
heat_transfer/test:meshgenerators/patch_sideset_generator.patch_sideset_generator_grid_2D 0.54 111.73

@joshuahansel joshuahansel marked this pull request as ready for review July 1, 2026 15:19
@joshuahansel

Copy link
Copy Markdown
Contributor Author

Nevermind my previous comments - I plan to close #33190 in favor of this one. This one is ready.

@joshuahansel joshuahansel merged commit 43a1d1a into idaholab:next Jul 1, 2026
57 checks passed
@joshuahansel joshuahansel deleted the graydiffuse-nonexistent-boundary branch July 1, 2026 16:00
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.

GrayDiffuseRadiation does not give error when non-existent boundary provided

3 participants