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

Namespace reorganization as agreed in the DD4hep developers meeting from 16th June 2017 #175

Merged
merged 15 commits into from Jun 19, 2017

Conversation

Projects
None yet
1 participant
@MarkusFrankATcernch
Contributor

MarkusFrankATcernch commented Jun 19, 2017

BEGINRELEASENOTES

DD4hep namespace reorganization

Re-organize namespaces according to the decisions of the DD4hep developers meeting from 16th June we have decided:

  1. all namespaces will be lower case and shorter
    • rename namespace DD4hep -> dd4hep
    • rename namespace DD4hep::DDRec -> dd4hep::rec
    • rename namespace DD4hep::Simulation -> dd4hep::sim
    • rename namespace XML -> xml and JSON -> json
    • rename all other namespaces according to this pattern
  2. The namespace DD4hep::Geometry:: will be incorporated into dd4hep::
  3. All utilities will be moved dd4hep::detail
  4. LCDD will be renamed to Detector and current Detector.h will be renamed to DetElement.h
  5. Examine if DDSegmentation can be incorporated into DDCore and make it volume aware
    • If this this cannot be achieved in whole place DDSegmentation into the right namespace

DDParsers

DDParsers are now a separate package. This does not make it yet standalone,
but it is at the same level as e.g. DDSeqmentation. Any librarian is
encouraged to externialize it fully.

ENDRELEASENOTES

@MarkusFrankATcernch MarkusFrankATcernch merged commit e744354 into AIDASoft:master Jun 19, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@MarkusFrankATcernch MarkusFrankATcernch deleted the MarkusFrankATcernch:ns_reorganization branch Jun 19, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment