-
Notifications
You must be signed in to change notification settings - Fork 31
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
Rename ORANGE construction files #1106
Rename ORANGE construction files #1106
Conversation
@@ -21,7 +21,7 @@ | |||
# include <fstream> | |||
# include <nlohmann/json.hpp> | |||
|
|||
# include "orange/construct/OrangeInputIO.json.hh" | |||
# include "orange/OrangeInputIO.json.hh" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This .cc file doesn't have any info about what version of ORANGE geometries are being updated. Though unrelated to this MR, consider adding a short description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, this just reads in a JSON file (which will translate any "old" parameter names, etc.) and writes it back out again. I'll add a brief in this file, but the app is actually documented in doc/main/usage.rst
.
This rearranges the ORANGE construction directory, moving the components used for "shape construction" (e.g. the CSG tree) into
orangeinp/
, and moving theOrangeInput
class to the main directory. The CSG renaming solves all the weird namespace issues ofusing namespace orange::csg
.