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

Miscellaneous mesh generator fixes and improvements #25086

Merged
merged 5 commits into from Aug 3, 2023

Conversation

GiudGiud
Copy link
Contributor

@GiudGiud GiudGiud commented Aug 2, 2023

More usable like this

Came from @Dlevario07 's meshing work

refs #25090 #22848 #24513

@GiudGiud GiudGiud self-assigned this Aug 2, 2023
@GiudGiud GiudGiud force-pushed the PR_daniel_misc branch 2 times, most recently from a4b508e to ab061a1 Compare August 3, 2023 00:12
@GiudGiud GiudGiud marked this pull request as ready for review August 3, 2023 00:30
@moosebuild
Copy link
Contributor

moosebuild commented Aug 3, 2023

Job Documentation on a52bf9d wanted to post the following:

View the site here

This comment will be updated on new commits.

@moosebuild
Copy link
Contributor

moosebuild commented Aug 3, 2023

Job Coverage on a52bf9d wanted to post the following:

Framework coverage

b3eabd #25086 a52bf9
Total Total +/- New
Rate 85.61% 85.61% -0.01% 44.44%
Hits 89665 89679 +14 16
Misses 15067 15079 +12 20

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

Warnings

  • framework new line coverage rate 44.44% is less than the suggested 90.0%

This comment will be updated on new commits.

framework/src/meshgenerators/ParsedCurveGenerator.C Outdated Show resolved Hide resolved
test/tests/meshgenerators/parsed_curve_generator/tests Outdated Show resolved Hide resolved
test/tests/meshgenerators/parsed_curve_generator/tests Outdated Show resolved Hide resolved
test/tests/meshgenerators/parsed_curve_generator/tests Outdated Show resolved Hide resolved
test/tests/meshgenerators/parsed_curve_generator/tests Outdated Show resolved Hide resolved
test/tests/meshgenerators/parsed_curve_generator/tests Outdated Show resolved Hide resolved
Better tests requirements
better error message

Co-authored-by: Alex Lindsay <alexander.lindsay@inl.gov>
@GiudGiud GiudGiud merged commit 3ad745c into idaholab:next Aug 3, 2023
45 of 46 checks passed
@GiudGiud GiudGiud deleted the PR_daniel_misc branch August 3, 2023 17:56
@GiudGiud
Copy link
Contributor Author

GiudGiud commented Aug 3, 2023

Thanks for the review!

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.

None yet

3 participants