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 plugin to assign Volume IDs to DetElements #1083

Merged
merged 1 commit into from
Mar 29, 2023

Conversation

MarkusFrankATcernch
Copy link
Contributor

@MarkusFrankATcernch MarkusFrankATcernch commented Mar 29, 2023

BEGINRELEASENOTES

  • Add plugin to assign VolumIDs to DetElements
    Normally this functionality id done by the VolumeManager. However, some clients prefer to not use the volume manager.
    With this plugin this basic functionality can be maintained.
  • Update plugin DD4hep_DetectorDump to optionally output the accumulated volume identifiers when traversing
    the DetElement tree.

To test both, invoke e.g.:

$> geoPluginRun -input ../DD4hep/DDDetectors/compact/SiD.xml \
     -plugin DD4hep_DetElementVolumeIDs -detector /world \
     -plugin DD4hep_DetectorDump -sensitive -volids

ENDRELEASENOTES

@MarkusFrankATcernch
Copy link
Contributor Author

@bcouturi Once merged, please check if this is what you wanted.

@MarkusFrankATcernch MarkusFrankATcernch enabled auto-merge (rebase) March 29, 2023 12:24
@MarkusFrankATcernch MarkusFrankATcernch merged commit 319b537 into AIDASoft:master Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant