Skip to content

Add type 38: 60GeV jets#4129

Merged
pinkenburg merged 1 commit intosPHENIX-Collaboration:masterfrom
pinkenburg:add-60GeV-jets
Jan 20, 2026
Merged

Add type 38: 60GeV jets#4129
pinkenburg merged 1 commit intosPHENIX-Collaboration:masterfrom
pinkenburg:add-60GeV-jets

Conversation

@pinkenburg
Copy link
Copy Markdown
Contributor

@pinkenburg pinkenburg commented Jan 20, 2026

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • [X ] 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, ...)

This PR adds the 60GeV pythia8 jet sample to the CreateFileList.pl as type 38. jenkins does not check this anyway: [skip-ci]

TODOs (if applicable)

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

Type 38: 60 GeV Pythia8 Jet Production Sample

Motivation / Context

This PR extends the sPHENIX production type catalog by adding support for a 60 GeV minimum transverse momentum pythia8 jet production sample. This complements the existing jet production types (5, 10, 15, 20, 30, 40, 50, 70 GeV) and fills a gap in the available high-pT jet simulation samples used for detector studies and physics analysis.

Key Changes

  • Production Type 38 Registration: Added new entry to %proddesc hash mapping: "38" => "JS pythia8 Jet ptmin = 60GeV"
  • Implementation Logic: Implements type 38 following the established pattern for jet production types:
    • Sets embedok = 1 to enable embedding capability
    • Initializes filename to pythia8_Jet60
    • Supports optional pileup embedding into Hijing backgrounds (pAu 0-10fm or AuAu central/default)
    • Applies pileup string augmentation unless nopileup flag is specified
  • File Modified: offline/framework/frog/CreateFileList.pl (+31/-1 lines)

Potential Risk Areas

  • None identified: This is a non-breaking additive change that strictly follows the existing code pattern for jet production types
  • IO/Reconstruction: No impact on reconstruction behavior or file formats
  • Performance: No performance implications

Notes

  • AI-Generated Content Alert: This summary is based on automated analysis of code patterns. Please verify that the filename convention pythia8_Jet60 correctly corresponds to the expected simulation configuration
  • Jenkins CI is intentionally skipped for this change (marked with [skip-ci])
  • The change maintains consistency with the existing architecture and requires no modifications to other components

Possible Future Improvements

  • If additional jet pT thresholds are needed, follow the same pattern established here
  • Consider documenting the available production types and their use cases in a central configuration reference

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

Adds production type 38 (JS pythia8 Jet ptmin = 60GeV) to CreateFileList.pl with embedding support and pileup handling logic. Enables Jet60-based simulations with optional embedding into Hijing scenarios.

Changes

Cohort / File(s) Summary
Production Type 38 Addition
offline/framework/frog/CreateFileList.pl
Introduces new production type 38 branch handling with embedok flag enabled, filenamestring initialized to pythia8_Jet60, and conditional embedding/pileup filename augmentation (pau, central, or default embedding scenarios vs. pp_pileupstring fallback). Updates %proddesc hash with corresponding descriptor "JS pythia8 Jet ptmin = 60GeV".

Possibly related PRs


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

@pinkenburg pinkenburg merged commit b7bf6cd into sPHENIX-Collaboration:master Jan 20, 2026
1 of 2 checks passed
@pinkenburg pinkenburg deleted the add-60GeV-jets branch January 20, 2026 17:43
@coderabbitai coderabbitai bot mentioned this pull request Feb 16, 2026
4 tasks
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