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

Write converted object and CSG metadata for debugging #1259

Merged
merged 14 commits into from
Jun 8, 2024

Conversation

sethrj
Copy link
Member

@sethrj sethrj commented Jun 3, 2024

This adds lots of delicious JSON diagnostic output to the ORANGE converter. The way to generate this currently (and to build the full ORANGE surface input at the same time) is to use the 'converter' unit test:

$ GDML=foo.gdml ./test/orange/orange_g4org_Converter --gtest_filter=ConverterTest.DISABLED_arbitrary --gtest_also_run_disabled_tests
# Generate the global universe (id 0)
$ ../scripts/user/orange-csg-to-dot.py -u 0 foo.gdml.csg.json | dot -Tpdf -o foo.pdf

tiletb-parent

@sethrj sethrj added enhancement New feature or request orange Work on ORANGE geometry engine labels Jun 3, 2024
@sethrj sethrj requested a review from elliottbiondo June 3, 2024 15:34
@sethrj sethrj force-pushed the metadata-output branch from 35f0985 to e085e98 Compare June 3, 2024 16:31
@sethrj sethrj force-pushed the metadata-output branch from 8bebcec to b0b07b8 Compare June 4, 2024 12:25
@sethrj
Copy link
Member Author

sethrj commented Jun 6, 2024

@elliottbiondo This one is also ready for review if you want

@elliottbiondo
Copy link
Contributor

(will do today)

@sethrj sethrj enabled auto-merge (squash) June 8, 2024 11:37
@sethrj sethrj merged commit 21a801a into celeritas-project:develop Jun 8, 2024
28 checks passed
@sethrj sethrj deleted the metadata-output branch August 1, 2024 17:27
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