Skip to content

Add DetectorVersionRunAction plugin and devcontainer setup#39

Open
wdconinc wants to merge 2 commits intomainfrom
detector-version-run-action
Open

Add DetectorVersionRunAction plugin and devcontainer setup#39
wdconinc wants to merge 2 commits intomainfrom
detector-version-run-action

Conversation

@wdconinc
Copy link
Contributor

What kind of PR is this?

Technical Change

What does this PR do?

Adds a new DD4hep Geant4RunAction plugin that stores the detector version information in run parameters, along with devcontainer configuration for consistent development environments.

Related PR(s) / Ticket(s)

None

Description

This PR introduces:

  1. A new DD4hep Geant4RunAction plugin that:

    • Retrieves the epic_version constant from DD4hep geometry
    • Stores the version information in DD4hep RunParameters
    • Provides proper error handling for missing constants
  2. Development container configuration:

    • Based on eicweb/eic_ci:nightly container
    • Pre-configured with DD4hep, ROOT, and EIC software stack
    • Setup VS Code extensions and development tools

Checklist

  • Code compiles without warnings
  • Unit tests pass
  • Added/updated necessary documentation
  • No new compiler warnings
  • PR description matches changes in code

Screenshots / Additional Information

None

@wdconinc
Copy link
Contributor Author

I had no authorship in this matter...

@wdconinc wdconinc force-pushed the detector-version-run-action branch from 019c4e9 to 929a59f Compare August 19, 2025 18:35
- Created DetectorVersionRunAction plugin to store epic_version in run parameters
- Updated CMakeLists.txt to include the new plugin
@wdconinc wdconinc force-pushed the detector-version-run-action branch from 929a59f to 4ef5b47 Compare August 19, 2025 19:36
- Replace custom UNUSED macro with standard C++ attribute
- Update both header and implementation files
- Remove macro definition
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