Skip to content

Conversation

@LinaresToine
Copy link
Contributor

Request of addition of new PD: https://cms-talk.web.cern.ch/t/new-primary-dataset-for-alcahcalisotrk-hlt-menu/124762

Questions:

  • Is the name AlCaHcalIsoTrk or HcalCalIsoTrkFromAlCaRaw?
  • Any alca producers? Or other parameter?

@saswatinandan

@mmusich
Copy link
Contributor

mmusich commented May 14, 2025

Hi @LinaresToine

Is the name AlCaHcalIsoTrk or HcalCalIsoTrkFromAlCaRaw?

the name of the new PD is AlCaHcalIsoTrk as you correctly have:

Any alca producers? Or other parameter?

the alca producer to be added is HcalCalIsoTrkFromAlCaRaw.
As for the other parameters you need at least to use AlCaHcalIsoTrk_Run3 as processing scenario, instead of ppScenario.
For the remaining ones I would propose to use the same as for the other AlCaRaw dataset:

do_reco=False,
raw_to_disk=True,
disk_node="T2_CH_CERN",

@mmusich
Copy link
Contributor

mmusich commented May 14, 2025

@LinaresToine one important piece of information, is that in order to test and deploy this PR you will need to move the cmssw release to 15_0_6 (that still doesn't exist).
The alca producer and scenario were introduced at cms-sw/cmssw#47975 which still is not in any release.

@JHiltbrand FYI

@JHiltbrand
Copy link

Thank you @mmusich for the ping and providing necessary information !

@LinaresToine
Copy link
Contributor Author

Thanks a lot @mmusich for clarifying all the important details, I have updated the PR.

I will now deploy a replay. Perhaps we cant really test the new PD since we don't have the corresponding stream as far as I know.

@mmusich
Copy link
Contributor

mmusich commented May 15, 2025

Perhaps we cant really test the new PD since we don't have the corresponding stream as far as I know.

yes, the stream and PD are not yet produced, so there's no data yet to test the PR.

@jeyserma jeyserma mentioned this pull request May 16, 2025
@LinaresToine LinaresToine merged commit 0f42cff into master May 16, 2025
alcaPPSScenario = "AlCaPPS_Run3"
hiTestppScenario = "ppEra_Run3_pp_on_PbPb_2023"
hiRawPrimeScenario = "ppEra_Run3_pp_on_PbPb_approxSiStripClusters_2023"
hltScoutingScenario = "hltScoutingEra_Run3_2025"

Choose a reason for hiding this comment

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

Hello,
at a first look this seems a mistake (was it done on purpose?)
Is a revert of the change requested in #5069 and applied in #5072

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.

5 participants