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

Add proto-universe and CSG Unit construction #1148

Merged
merged 16 commits into from
Mar 15, 2024

Conversation

sethrj
Copy link
Member

@sethrj sethrj commented Mar 13, 2024

This is the next-to-last step for programmatically building an ORANGE geometry from CSG objects. It defines a "proto-universe" interface for defining individual universes (aka "scenes") and building them. For testing and external integration with SCALE, the UnitProto class has a helper function for returning a CsgUnit which makes testing a cinch.

The following PR will take the constructed CSG unit, build a UniverseInput, and actually construct an OrangeParams object for tracking.

@sethrj sethrj added enhancement New feature or request orange Work on ORANGE geometry engine labels Mar 13, 2024
@sethrj sethrj requested a review from elliottbiondo March 13, 2024 12:44

//---------------------------------------------------------------------------//
/*!
* Construct a universe for building ORANGE geometry.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

an ORANGE geometry

src/orange/orangeinp/ProtoInterface.hh Outdated Show resolved Hide resolved
src/orange/orangeinp/UnitProto.hh Outdated Show resolved Hide resolved
src/orange/orangeinp/UnitProto.hh Outdated Show resolved Hide resolved
/*!
* Get a list of all daughter protos.
*/
auto UnitProto::daughters() const -> VecProto
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if this is enough work to cross into "calc_daughters"

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe... maybe get_daughters? I'll revisit this.

src/orange/orangeinp/UnitProto.cc Outdated Show resolved Hide resolved
@sethrj sethrj enabled auto-merge (squash) March 15, 2024 22:18
@sethrj sethrj merged commit 291393e into celeritas-project:develop Mar 15, 2024
21 checks passed
@sethrj sethrj deleted the build-unit branch March 16, 2024 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request orange Work on ORANGE geometry engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants