moose icon indicating copy to clipboard operation
moose copied to clipboard

XYDelaunayGenerator with Optimization Example

Open maxnezdyur opened this issue 1 year ago • 6 comments

closes #27722

maxnezdyur avatar May 28 '24 18:05 maxnezdyur

Job Documentation on 2de1910 wanted to post the following:

View the site here

This comment will be updated on new commits.

moosebuild avatar May 28 '24 23:05 moosebuild

I would suggest implement a feature that allows reading interior points from a file, which would be particularly beneficial for managing extensive lists, potentially generated by a script.

miaoyinb avatar May 30 '24 03:05 miaoyinb

Do you only need this list at the time of mesh generation? Or should the mesh be modified later

The Positions system would be good to use here. We would just have Positions be created earlier.

GiudGiud avatar May 30 '24 04:05 GiudGiud

@GiudGiud

The Positions system would be good to use here. We would just have Positions be created earlier.

Are we going to create problems by doing this since some position objects rely on the mesh? Like the ElementCentroidPositions? Positions is a GeneralUserObject, can we even run any use object before mesh creation or would this require us to add another execute_on flag?

maxnezdyur avatar May 30 '24 13:05 maxnezdyur

There would be some work to do for sure

UOs are created in the problem which does not exist yet at that point

GiudGiud avatar May 30 '24 13:05 GiudGiud

Job Coverage on 2de1910 wanted to post the following:

Framework coverage

ca913d #27723 2de191
Total Total +/- New
Rate 85.08% 85.08% +0.00% 100.00%
Hits 104117 104133 +16 17
Misses 18265 18265 - 0

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

This comment will be updated on new commits.

moosebuild avatar Jun 13 '24 15:06 moosebuild

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

View the site here

This comment will be updated on new commits.

moosebuild avatar Nov 03 '24 19:11 moosebuild

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

Framework coverage

af6bed #27723 e57852
Total Total +/- New
Rate 85.09% 85.10% +0.00% 100.00%
Hits 106734 106755 +21 22
Misses 18696 18696 - 0

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

This comment will be updated on new commits.

moosebuild avatar Nov 04 '24 16:11 moosebuild

@roystgnr if you d like to re-review otherwise I would merge this

GiudGiud avatar Nov 05 '24 22:11 GiudGiud

Testing failures unrelated: openmpi classic failures on multiapps coupling and some network issue on one of 2 HPC platforms

GiudGiud avatar Nov 27 '24 20:11 GiudGiud

Few things here:

  • Tests weren't ran on this PR in over 3 weeks, please consider invalidating before merge
  • @maxnezdyur please consider adding valgrind testing to these optimization PRs. These tests tend to be more heavy than not and we've had multiple occurrences now where we've need to make one of the valgrind tests heavy

loganharbour avatar Nov 28 '24 03:11 loganharbour