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

pixel mc gain calibration scheme: new GTs for MC Run3, modified Clusterizer conditions for Run3 #29333

Merged
merged 4 commits into from Apr 18, 2020

Conversation

dkotlins
Copy link
Contributor

@dkotlins dkotlins commented Mar 29, 2020

PR description:

Modify the scheme of the pixel gain calibration. Instead of applying the vcal calibration in the
clusterizer include it already in the gain calibration payload. Like this the vcal calibration can
be easily adjusted for each new gain calibration.
This is the first step in which we only change the MC gain calibrations. It should apply to any future Run3 simulations. Data from Run1/2 and older, none Run3 MC, will not be affected.

To be applied for Run 3 MC & Data. No code changes, the only changes are:

  1. new GTs for Run3 MC which include the new gain calibration values,
  2. changes in the clusterizer parameters for Run 3, for the VCAL calibration use slope=1 and offset =0.

PR validation:

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

No backport required.

The changes have been verified using Run3 simulations.

@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29333/14411

Code check has found code style and quality issues which could be resolved by applying following patch(s)

@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29333/14420

Code check has found code style and quality issues which could be resolved by applying following patch(s)

@mmusich
Copy link
Contributor

mmusich commented Mar 30, 2020

@dkotlins this PR has code style and quality issues. Can you please resolve them as suggested at #29333 (comment)?
In alternative you can run scram b code-format and scram b code-checks in your local development area and then push the changes.
Otherwise the PR integration tests cannot be performed.

@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-29333/14422

@mmusich
Copy link
Contributor

mmusich commented Apr 17, 2020

Can AlCa please review?
@tlampen @tocheng @christopheralanwest

@tocheng
Copy link
Contributor

tocheng commented Apr 17, 2020

+1

Sorry for being late.

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo (and backports should be raised in the release meeting by the corresponding L2)

@silviodonato
Copy link
Contributor

+1
the differences in DQM plots are explained in #29333 (comment)

@cmsbuild cmsbuild merged commit c8df96f into cms-sw:master Apr 18, 2020
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Jul 4, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Jul 4, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Oct 8, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Oct 19, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Oct 20, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Oct 23, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Nov 6, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Nov 16, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard added a commit to cms-patatrack/cmssw that referenced this pull request Nov 27, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Dec 25, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Dec 29, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Dec 29, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
fwyzard pushed a commit to cms-patatrack/cmssw that referenced this pull request Dec 29, 2020
Update the Patatrack code following cms-sw#29333:

Modify the scheme of the pixel gain calibration: instead of applying the VCal calibration in the
clusterizer include it already in the gain calibration payload.
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.

None yet

8 participants