Elliott Biondo comments

Results 15 comments of


                                            Elliott Biondo

Add oriented bounding zone class to be used for safety distance calculations

@sethrj this is ready for your review

Add oriented bounding zone class to be used for safety distance calculations

(minor floating point issue causing one test failure in one build configuration, will fix in morning)

Add oriented bounding zone class to be used for safety distance calculations

@sethrj this is ready for another look

Add oriented bounding zone class to be used for safety distance calculations

@sethrj this should be ready

Require a minimum number of volumes to build a BIH tree

Draft until this is refactor to instead set a maximum leaf size

Require a minimum number of volumes to build a BIH tree

Closing this MR; the minimum leaf size approach used here is superior: #1930 Determining this parameter will done with benchmark problems on ANL JLSE per https://github.com/celeritas-project/regression/issues/3.

Convert Geant4 geometry to ORANGE

Support added for *unaccelerated* `MultiUnion` via https://github.com/celeritas-project/celeritas/pull/1987. Checking all `MultiUnion` boxes.

Use infix evaluation

If infix/postfix could be toggled as a runtime option that would side-step the issue with the orange test data, correct?

Add options for constructing an ORANGE geometry

@sethrj can you take a preliminary look at this? We already have `g4org::Options` -> `InputBuilder::Options`, so the addition of `OrangeInput::ConstructionOptions` feels suspicious.

Reverse build order of protos

@sethrj this is ready for a first look. As best as I can tell we don't have any unit tests that actually resulted in initial guesses for bboxes affecting the...