moose icon indicating copy to clipboard operation
moose copied to clipboard

Update Optimization Module and remove duplicative code

Open maxnezdyur opened this issue 1 year ago • 1 comments

closes #26564

maxnezdyur avatar Jan 16 '24 13:01 maxnezdyur

Job Documentation on 93462ef wanted to post the following:

View the site here

This comment will be updated on new commits.

moosebuild avatar Jan 16 '24 15:01 moosebuild

This pull request has been automatically marked as stale because it has not had recent activity in the last 100 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Apr 29 '24 05:04 github-actions[bot]

Will work on soon.

maxnezdyur avatar Apr 29 '24 10:04 maxnezdyur

Job Coverage on 93462ef wanted to post the following:

Framework coverage

Coverage did not change

Modules coverage

Optimization

b3efaa #26565 93462e
Total Total +/- New
Rate 89.18% 88.61% -0.57% 78.95%
Hits 2003 1976 -27 15
Misses 243 254 +11 4

Diff coverage report

Full coverage report

Full coverage reports

Reports

Warnings

  • optimization new line coverage rate 78.95% is less than the suggested 90.0%

This comment will be updated on new commits.

moosebuild avatar May 07 '24 15:05 moosebuild

@maxnezdyur can close all of my comments, you have answered all of my questions. Great work. @GiudGiud I reviewed and it looks good, would you have time to review and merge?

lynnmunday avatar May 07 '24 21:05 lynnmunday

I ll do it soon!

GiudGiud avatar May 07 '24 22:05 GiudGiud

@lynnmunday note that because you edited my username into the previous comment, I never actually got a notification with that text

GiudGiud avatar May 07 '24 22:05 GiudGiud