Skip to content

Adding truth info for FCC-ee ILD model, add vertex resolutions as inp… #188

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.

Sign up for GitHub

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

Open
armin-ilg wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Adding truth info for FCC-ee ILD model, add vertex resolutions as inp… #188

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
0422a8d
Adding truth info for FCC-ee ILD model, add vertex resolutions as inp…
armin-ilg Jul 20, 2026
303def9
Update StandardConfig/production/ILDReconstruction.py
armin-ilg Aug 12, 2026
79fad74
Address comments, remove truth matching for FCC-ee as this will be ha…
armin-ilg Aug 14, 2026
09b8da3
Merge branch 'master' into master
armin-ilg Aug 14, 2026
6becb42
Fix missing definition of algorithm
tmadlener Aug 14, 2026
c0cf856
Merge branch 'master' into truth-info-fcc
tmadlener Aug 19, 2026
3b83b87
Fix formatting
tmadlener Aug 19, 2026
ab78066
Merge branch 'master' into truth-info-fcc
tmadlener Aug 26, 2026
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
93 changes: 93 additions & 0 deletions StandardConfig/production/ILDReconstruction.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -163,6 +163,34 @@
action="store_true",
)

parser.add_argument(
Comment thread
armin-ilg marked this conversation as resolved.
"--VXDBarrelDigitiserResolutionU",
action="store",
default=["0.003"],
help="Resolution in U direction for VXD Barrel digitisation", nargs="+"
)

parser.add_argument(
"--VXDBarrelDigitiserResolutionV",
action="store",
default=["0.003"],
help="Resolution in V direction for VXD Barrel digitisation", nargs="+"
)

parser.add_argument(
"--VXDEndcapDigitiserResolutionU",
action="store",
default=["0.003"],
help="Resolution in U direction for VXD Endcap digitisation", nargs="+"
)

parser.add_argument(
"--VXDEndcapDigitiserResolutionV",
action="store",
default=["0.003"],
help="Resolution in V direction for VXD Endcap digitisation", nargs="+"
)


def get_compact_file_path(detector_model: str):
"""returns the compact file path to a specified detector model starting with the path stored in the 'K4GEO' environment variable"""
Expand Down Expand Up @@ -241,6 +269,10 @@ def get_cms_energy_config(
"CONSTANTS": CONSTANTS,
"cms_energy_config": cms_energy_config,
"using_particle_gun": reco_args.usingParticleGun,
"VXDBarrelDigitiserResolutionU": reco_args.VXDBarrelDigitiserResolutionU,
"VXDBarrelDigitiserResolutionV": reco_args.VXDBarrelDigitiserResolutionV,
"VXDEndcapDigitiserResolutionU": reco_args.VXDEndcapDigitiserResolutionU,
"VXDEndcapDigitiserResolutionV": reco_args.VXDEndcapDigitiserResolutionV,
},
)

Expand Down Expand Up @@ -295,6 +327,67 @@ def get_cms_energy_config(

if not is_FCCee_model:
sequenceLoader.load("HighLevelReco/HighLevelReco")
else:
MyRecoMCTruthLinker = MarlinProcessorWrapper("MyRecoMCTruthLinker")
MyRecoMCTruthLinker.ProcessorType = "RecoMCTruthLinker"
MyRecoMCTruthLinker.Parameters = {
"CalohitMCTruthLinkName": ["CalohitMCTruthLink"],
"ClusterCollection": ["PandoraClusters"],
"ClusterMCTruthLinkName": ["ClusterMCTruthLink"],
"FullRecoRelation": ["true"],
"KeepDaughtersPDG": ["22", "111", "310", "13", "211", "321"],
"MCParticleCollection": ["MCParticle"],
"MCParticlesSkimmedName": ["MCParticlesSkimmed"],
"MCTruthClusterLinkName": ["MCTruthClusterLink"],
"MCTruthRecoLinkName": ["MCTruthRecoLink"],
"MCTruthTrackLinkName": ["MCTruthMarlinTrkTracksLink"],
"RecoMCTruthLinkName": ["RecoMCTruthLink"],
"RecoParticleCollection": ["PandoraPFOs"],
"SimCaloHitCollections": [
"BeamCalCollection",
"LHCalCollection",
"LumiCalCollection",
CONSTANTS["ECalSimHitCollections"],
CONSTANTS["HCalSimHitCollections"],
"YokeBarrelCollection",
"YokeEndcapsCollection",
],
"SimCalorimeterHitRelationNames": [
"EcalBarrelRelationsSimRec",
"EcalEndcapRingRelationsSimRec",
"EcalEndcapsRelationsSimRec",
"HcalBarrelRelationsSimRec",
"HcalEndcapRingRelationsSimRec",
"HcalEndcapsRelationsSimRec",
"RelationLHcalHit",
"RelationMuonHit",
"RelationLcalHit",
"RelationBCalHit",
],
"SimTrackerHitCollections": [
"VXDCollection",
"SITCollection",
"FTD_PIXELCollection",
"FTD_STRIPCollection",
Comment thread
armin-ilg marked this conversation as resolved.
Outdated
"TPCCollection",
"SETCollection",
],
"TrackCollection": ["MarlinTrkTracks"],
"TrackMCTruthLinkName": ["MarlinTrkTracksMCTruthLink"],
"TrackerHitsRelInputCollections": [
"VXDTrackerHitRelations",
"SITTrackerHitRelations",
"FTDPixelTrackerHitRelations",
"FTDSpacePointRelations",
Comment thread
armin-ilg marked this conversation as resolved.
Outdated
"TPCTrackerHitRelations",
"SETSpacePointRelations",
],
"UseTrackerHitRelations": ["true"],
"UsingParticleGun": ["true"],
}
algList.append(MyRecoMCTruthLinker)
Comment thread
armin-ilg marked this conversation as resolved.
Outdated



if not reco_args.noPFO:
MyPfoAnalysis = MarlinProcessorWrapper("MyPfoAnalysis")
Expand Down
12 changes: 6 additions & 6 deletions StandardConfig/production/Tracking/TrackingDigi_FCCeeMDI.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,23 +10,23 @@
VertexBarrelDigitiser.ProcessorType = "DDPlanarDigiProcessor"
VertexBarrelDigitiser.Parameters = {
"IsStrip": ["false"],
"ResolutionU": ["0.003", "0.003", "0.003", "0.003", "0.003", "0.003"],
"ResolutionV": ["0.003", "0.003", "0.003", "0.003", "0.003", "0.003"],
"ResolutionU": [str(x) for x in VXDBarrelDigitiserResolutionU],
"ResolutionV": [str(x) for x in VXDBarrelDigitiserResolutionV],
Comment thread
armin-ilg marked this conversation as resolved.
"SimTrackHitCollectionName": ["VertexBarrelCollection"],
"SimTrkHitRelCollection": ["VertexBarrelTrackerHitRelations"],
"SubDetectorName": ["VertexBarrel"],
"SubDetectorName": ["Vertex"],
"TrackerHitCollectionName": ["VertexBarrelTrackerHits"],
}

VertexEndcapDigitiser = MarlinProcessorWrapper("VertexEndcapDigitiser")
VertexEndcapDigitiser.ProcessorType = "DDPlanarDigiProcessor"
VertexEndcapDigitiser.Parameters = {
"IsStrip": ["false"],
"ResolutionU": ["0.003", "0.003", "0.003", "0.003", "0.003", "0.003"],
"ResolutionV": ["0.003", "0.003", "0.003", "0.003", "0.003", "0.003"],
"ResolutionU": [str(x) for x in VXDEndcapDigitiserResolutionU],
"ResolutionV": [str(x) for x in VXDEndcapDigitiserResolutionV],
"SimTrackHitCollectionName": ["VertexEndcapCollection"],
"SimTrkHitRelCollection": ["VertexEndcapTrackerHitRelations"],
"SubDetectorName": ["VertexEndcap"],
"SubDetectorName": ["Vertex"],
"TrackerHitCollectionName": ["VertexEndcapTrackerHits"],
}

Expand Down
36 changes: 18 additions & 18 deletions StandardConfig/production/Tracking/TrackingReco_FCCeeMDI.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

from py_utils import encode_CT_steps_dict_to_legacy_list

CT_MAX_DIST = "0.03" # RANDOM VALUE COPYIED FROM CLDRECO
CT_MAX_DIST = "0.05" # RANDOM VALUE COPYIED FROM CLDRECO
Comment thread
armin-ilg marked this conversation as resolved.
Outdated
MCPartColName = ["MCParticle"] # MCParticleCollectionName
VertexBarrelHitCollectionNames = ["VertexBarrelTrackerHits"]
VertexEndcapHitCollectionNames = ["VertexEndcapTrackerHits"]
Expand Down Expand Up @@ -38,15 +38,15 @@
"TrackStartsInnerDist": ["25"],
"TrackSystemName": ["DDKalTest"],
"VXDHitCollection": ["VertexBarrelTrackerHits"],
"VXDDetectorName": ["VertexBarrel"],
"VXDDetectorName": ["Vertex"],

@tmadlener tmadlener Aug 26, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't realized this before, but this has no effect as the next line overwrites the same property with "VertexBarrel" again.

@armin-ilg armin-ilg Aug 27, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, this must have been introduced somewhere along the way, I'll fix it.

"pickUpSiHits": ["true"],
}

MyConformalTracking = MarlinProcessorWrapper("MyConformalTracking")
MyConformalTracking.ProcessorType = "ConformalTrackingV2"
conformal_tracking_steps_config = {
# Based on CLD's Reconstruction in CLDConfig
"VertexBarrel": {
"Vertex": {
"collections": ["VertexBarrelTrackerHits"],
"params": {
"MaxCellAngle": 0.01,
Expand All @@ -60,7 +60,7 @@
"flags": ["HighPTFit", "VertexToTracker"],
"functions": ["CombineCollections", "BuildNewTracks"],
},
"VertexEncap": {
"Vertex": {
"collections": ["VertexEndcapTrackerHits"],
"params": {
"MaxCellAngle": 0.01,
Expand Down Expand Up @@ -92,20 +92,20 @@
"flags": ["HighPTFit", "VertexToTracker", "RadialSearch"],
"functions": ["CombineCollections", "BuildNewTracks"],
},
# "LowerCellAngle2": {
# "collections": "",
# "params": {
# "MaxCellAngle": 0.1,
# "MaxCellAngleRZ": 0.1,
# "Chi2Cut": 2000,
# "MinClustersOnTrack": 4,
# "MaxDistance": CT_MAX_DIST,
# "SlopeZRange": 10.0,
# "HighPTCut": 10.0,
# },
# "flags": ["HighPTFit", "VertexToTracker", "RadialSearch"],
# "functions": ["BuildNewTracks","SortTracks"],
# },
"LowerCellAngle2": {
"collections": [""],
"params": {
"MaxCellAngle": 0.1,
"MaxCellAngleRZ": 0.1,
"Chi2Cut": 2000,
"MinClustersOnTrack": 4,
"MaxDistance": CT_MAX_DIST,
"SlopeZRange": 10.0,
"HighPTCut": 10.0,
},
"flags": ["HighPTFit", "VertexToTracker", "RadialSearch"],
"functions": ["BuildNewTracks","SortTracks"],
},
Comment thread
armin-ilg marked this conversation as resolved.
"Tracker": {
"collections": ["InnerTrackerBarrelHits", "InnerTrackerEndcapHits"],
"params": {
Expand Down