feat: add setter for cluster map name

[osbornjd]

Adds a cluster map name setter so that only the seed DST can be read in

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work for users)
• Requiring change in macros repository (Please provide links to the macros pull request in the last section)
• I am a member of GitHub organization of sPHENIX Collaboration, EIC, or ECCE (contact Chris Pinkenburg to join)

What kind of change does this PR introduce? (Bug fix, feature, ...)

TODOs (if applicable)

Links to other PRs in macros and calibration repositories (if applicable)

Summary

Motivation / Context
This PR adds configurability for the cluster container lookup in TrackSeedTrackMapConverter, enabling flexible selection of cluster data sources during track seed to track map conversion. Previously, the cluster container was hardcoded to "TRKR_CLUSTER", limiting users' ability to process alternative cluster containers (e.g., different seed DST inputs).

Key Changes

• Added public setter method setClusterMapName(const std::string& name) to TrackSeedTrackMapConverter class
• Introduced private member variable m_clusterMapName with default value "TRKR_CLUSTER" to maintain backward compatibility
• Updated getNodes() method to use m_clusterMapName for cluster container lookup instead of hardcoded string
• Improved error message to report the requested cluster container name, aiding debugging

Potential Risk Areas

• Reconstruction behavior: Users must ensure the specified cluster container exists in the DST, or the conversion will abort with an event error; silent failures are prevented by explicit error handling
• Backward compatibility: Default behavior is preserved ("TRKR_CLUSTER"), so existing configurations unaffected
• Configuration dependency: New configuration option adds an implicit dependency on correct DST structure that callers must manage

Possible Future Improvements

• Consider allowing multiple cluster containers or a list of fallback options if one is unavailable
• Add documentation/examples of supported cluster container names in coresoftware wiki
• Consider similar configurability for other hardcoded container names if patterns emerge

Note on AI-generated analysis: This summary is based on AI analysis of code changes. Readers should verify details against actual PR code and consult with the author regarding specific use cases and DST structure requirements.

[coderabbitai]

📝 Walkthrough

Walkthrough

TrackSeedTrackMapConverter now uses a configurable cluster map name instead of a hard-coded "TRKR_CLUSTER" string. A private member variable with default value is introduced, along with a public setter method to allow external configuration of the cluster container lookup key.

Changes

Cohort / File(s)	Summary
TrackSeedTrackMapConverter Implementation offline/packages/TrackingDiagnostics/TrackSeedTrackMapConverter.h, TrackSeedTrackMapConverter.cc	Introduced configurable cluster map name via new private member m_clusterMapName (default: "TRKR_CLUSTER") and public setter setClusterMapName(). Replaced hard-coded cluster container lookup key with the configurable member; updated error messages to reflect actual container names being requested.

Possibly related PRs

• feat: make cluster map setters for job c #4099: Implements similar configurable cluster map names across other tracking diagnostics modules using the same approach.

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sphenix-jenkins-ci]

Build & test report

Report for commit 83a66dbe76cb5dcb3eaccbe002fac9f171bd2321:

• builds and tests overall are SUCCESS.
• Build with configuration of alma9.2-gcc-14.2.0 / clang is SUCCESS, 📊clang report (full)/(new), build log
• Build with configuration of alma9.2-gcc-14.2.0 / new is SUCCESS, 📊Compiler report (full)/(new), build log
  • Generating DST and readback: build is SUCCESS
  • Calorimeter QA: build is SUCCESS, 📊 trends
    • 📊 QA-calorimeter for e- at p_T=4GeV : combined Chi2/nDoF = 27.45 / 72, and combined p-Value = 1
    • 📊 QA-calorimeter for pi+ at p_T=30GeV : combined Chi2/nDoF = 41.6183 / 72, and combined p-Value = 0.998442
    • 📊 QA-calorimetric-jet for e- at p_T=4GeV : combined Chi2/nDoF = 0.4559 / 42, and combined p-Value = 1
    • 📊 QA-calorimetric-jet for pi+ at p_T=30GeV : combined Chi2/nDoF = 14.2531 / 42, and combined p-Value = 0.999981
  • Tracking QA with Distortions: build is SUCCESS, 📊 trends
    • 📊 QA-distortions
  • Tracking QA at low occupancy: build is SUCCESS, 📊 trends
    • 📊 QA-Intt : combined Chi2/nDoF = -0 / 72, and combined p-Value = 1
    • 📊 QA-Micromegas : combined Chi2/nDoF = -0 / 20, and combined p-Value = 1
    • 📊 QA-Mvtx : combined Chi2/nDoF = -0 / 54, and combined p-Value = 1
    • 📊 QA-Tpc : combined Chi2/nDoF = -0 / 56, and combined p-Value = 1
    • 📊 QA-tracking : combined Chi2/nDoF = -0 / 56, and combined p-Value = 1
    • 📊 QA-vertexing : combined Chi2/nDoF = 30 / 112, and combined p-Value = 1
  • Tracking QA for Pythia D0-jet: build is SUCCESS, 📊 trends
    • 📊 QA-Intt : combined Chi2/nDoF = -0 / 72, and combined p-Value = 1
    • 📊 QA-Micromegas : combined Chi2/nDoF = -0 / 20, and combined p-Value = 1
    • 📊 QA-Mvtx : combined Chi2/nDoF = -0 / 54, and combined p-Value = 1
    • 📊 QA-Tpc : combined Chi2/nDoF = 2.15224e-09 / 56, and combined p-Value = 1
    • 📊 QA-tracking : combined Chi2/nDoF = -0 / 50, and combined p-Value = 1
    • 📊 QA-vertexing : combined Chi2/nDoF = 30 / 112, and combined p-Value = 1
  • Tracking QA from run2pp 53877: build is SUCCESS, 📊 trends
    • 📊 QA-intt : combined Chi2/nDoF = -0 / 18, and combined p-Value = 1
    • 📊 QA-micromegas : combined Chi2/nDoF = -0 / 6, and combined p-Value = 1
    • 📊 QA-mvtx : combined Chi2/nDoF = -0 / 18, and combined p-Value = 1
    • 📊 QA-siliconseeds : combined Chi2/nDoF = -0 / 20, and combined p-Value = 1
    • 📊 QA-tpc : combined Chi2/nDoF = -0 / 48, and combined p-Value = 1
    • 📊 QA-tpcseeds : combined Chi2/nDoF = -0 / 28, and combined p-Value = 1
    • 📊 QA-trackfit : combined Chi2/nDoF = -0 / 32, and combined p-Value = 1
  • system alma9.2-gcc-14.2.0, build new: run the default sPHENIX macro: build is SUCCESS, output, trends 📊
  • system alma9.2-gcc-14.2.0, build new: run the default CaloProduction/Fun4All_Year2.C macro: build is SUCCESS, output, trends 📊
  • system alma9.2-gcc-14.2.0, build new: run the default StreamingProduction/Fun4All_Stream_Combiner.C macro: build is SUCCESS, output, trends 📊
  • system alma9.2-gcc-14.2.0, build new: Valgrind test of CaloProduction/Fun4All_Year2.C: build is SUCCESS, 📊valgrind report, trends 📊
  • system alma9.2-gcc-14.2.0, build new: Valgrind test of StreamingProduction/Fun4All_Stream_Combiner.C: build is SUCCESS, 📊valgrind report, trends 📊
  • system alma9.2-gcc-14.2.0, build new: run the overlap check for sPHENIX macro: build is SUCCESS, output
  • system alma9.2-gcc-14.2.0, build new: Valgrind test: build is SUCCESS, 📊valgrind report, trends 📊
• Build with configuration of alma9.2-gcc-14.2.0 / scan is SUCCESS, 📊scan-build report (full)/(new), build log
• clang-tidy is SUCCESS, 📊clang-tidy report (full)/(new)
• cpp-check is SUCCESS, 📊cppcheck report (full)/(new)

---

Automatically generated by sPHENIX Jenkins continuous integration