Skip to content
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

New workflows for displaced muon (Dxy=100) and single ele/mu with pt… #33602

Closed
wants to merge 1 commit into from

Conversation

rgoldouz
Copy link
Contributor

@rgoldouz rgoldouz commented May 3, 2021

… [1.5-8] is added to be used in relval production for phase2 L1 track trigger studies

PR description:

PR validation:

if this PR is a backport please specify the original PR and why you need to backport that PR:

Before submitting your pull requests, make sure you followed this checklist:

… [1.5-8] is added to be used in relval production for phase2 L1 track trigger studies
@cmsbuild
Copy link
Contributor

cmsbuild commented May 3, 2021

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33602/22421

@cmsbuild
Copy link
Contributor

cmsbuild commented May 3, 2021

A new Pull Request was created by @rgoldouz (Reza Goldouzian) for master.

It involves the following packages:

Configuration/PyReleaseValidation

@jordan-martins, @chayanit, @wajidalikhan, @kpedro88, @cmsbuild, @srimanob can you please review it and eventually sign? Thanks.
@makortel, @Martin-Grunewald, @fabiocos, @slomeo this is something you requested to watch as well.
@silviodonato, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

@rgoldouz
Copy link
Contributor Author

rgoldouz commented May 3, 2021

Hello,

The parameters in the following gen card is not appropriate for our studies in phase2 L1 track triggers.
https://github.com/cms-sw/cmssw/blob/master/Configuration/Generator/python/FlatRandomPtAndDxyGunProducer_cfi.py
In addition we needed dedicated single muon and electron samples with low pt range.

Each time we asked for these relval samples, relval people had to change those parameters by hand and it delays our requests and it was difficult for them to follow our requests.

In my recent request for relval samples, they suggested to add new cards and add them to the workflow list. then I can easily provide them the workflow ID and they can submit my requests.
https://its.cern.ch/jira/browse/PDMVRELVALS-116

after discussion with the Gen group, new cards are merged
#33499

In this PR I just added our new merged cards to the workflows. After merging these changes I can ask for new relval samples.

Please let me know if you have any comment/quesstion.

Thanks a lot,
Reza

@chayanit
Copy link

chayanit commented May 3, 2021

Hello @rgoldouz , I'm taking care of adding your new GEN fragments to runTheMatrix don't worry. Could you please close this PR? Usually one should add new workflows at the end to not ruin the existing order of workflows.

#33461 FYI.

@rgoldouz
Copy link
Contributor Author

rgoldouz commented May 3, 2021

Hi chayanit,

Sure, thanks.

Reza

@rgoldouz rgoldouz closed this May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants