Skip to content

Add SimPulse data type for storing simulated pulses pre-digitization #106

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

Merged
merged 16 commits into from
Apr 14, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ project(EDM4EIC
LANGUAGES CXX)

SET( ${PROJECT_NAME}_VERSION_MAJOR 8 )
SET( ${PROJECT_NAME}_VERSION_MINOR 0 )
SET( ${PROJECT_NAME}_VERSION_MINOR 1 )
SET( ${PROJECT_NAME}_VERSION_PATCH 0 )
SET( ${PROJECT_NAME}_VERSION "${${PROJECT_NAME}_VERSION_MAJOR}.${${PROJECT_NAME}_VERSION_MINOR}.${${PROJECT_NAME}_VERSION_PATCH}" )

Expand Down
25 changes: 23 additions & 2 deletions edm4eic.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,9 @@
## Patch level changes are required to be schema invariant.
##
## If there are schema version changes that can be evolved, see the podio documentation
## for an example: https://github.com/hegner/podio/blob/master/tests/schema_evolution.yaml
## for an example: https://github.com/AIDASoft/podio/tree/master/tests/schema_evolution
##
schema_version: 800
schema_version: 801

options :
# should getters / setters be prefixed with get / set?
Expand Down Expand Up @@ -203,6 +203,27 @@ datatypes:
- float floatData // Iff elementType==1, values are stored here
- int64_t int64Data // Iff elementType==7, values are stored here

## ==========================================================================
## Simulation info
## ==========================================================================

edm4eic::SimPulse:
Description: "Simulated pulse prior to digitization."
Author: "D. Anderson, S. Gardner, S. Joosten., D. Kalinkin"
Members:
- uint64_t cellID // ID of the readout cell for this pulse.
- float integral // Total pulse integral in relevant units.
- edm4hep::Vector3f position // Position the pulse is evaluated in world coordinates [mm].
- float time // Start time for the pulse in [ns].
- float interval // Time interval between amplitude values [ns].
VectorMembers:
- float amplitude // Pulse amplitude in relevant units, sum of amplitude values equals integral
OneToManyRelations:
- edm4hep::SimCalorimeterHit calorimeterHits // SimCalorimeterHits used to create this pulse
- edm4hep::SimTrackerHit trackerHits // SimTrackerHits used to create this pulse
- edm4eic::SimPulse pulses // SimPulses used to create this pulse
- edm4hep::MCParticle particles // MCParticle that caused the pulse

## ==========================================================================
## Particle info
## ==========================================================================
Expand Down