Skip to content

Add some missing RBParameters APIs for "extra" parameters#3576

Merged
jwpeterson merged 3 commits intolibMesh:develfrom
jwpeterson:get_extra_step_value
Jun 6, 2023
Merged

Add some missing RBParameters APIs for "extra" parameters#3576
jwpeterson merged 3 commits intolibMesh:develfrom
jwpeterson:get_extra_step_value

Conversation

@jwpeterson
Copy link
Copy Markdown
Member

Also add some unit tests so hopefully the coverage is not 0%.

Initially I did not think I would need full multi-step support for
getting the "extra" parameters, but it turns out we do, so I'm adding
it now.
Again, we already had this support for "normal" parameters, so makes
sense to add it for "extra" parameters as well.
@moosebuild
Copy link
Copy Markdown

Job Coverage on 8710a46 wanted to post the following:

Coverage

7aa532 #3576 8710a4
Total Total +/- New
Rate 60.25% 60.25% +0.01% 77.78%
Hits 49412 49422 +10 7
Misses 32606 32605 -1 2

Diff coverage report

Full coverage report

Warnings

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

This comment will be updated on new commits.

@jwpeterson
Copy link
Copy Markdown
Member Author

Same failure in mesh/preparedness.test that we've been seeing.

@jwpeterson jwpeterson merged commit 22174ac into libMesh:devel Jun 6, 2023
@jwpeterson jwpeterson deleted the get_extra_step_value branch June 6, 2023 14:18
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.

2 participants