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

Adding CSA14 GTs and enabling pre-mixing in autoCond.py #2868

Merged
merged 2 commits into from Mar 14, 2014

Conversation

diguida
Copy link
Contributor

@diguida diguida commented Mar 14, 2014

Updates autoCond.py by adding the GTs for the pessimistic scenario (aka 50 ns) in CSA14, and changing the GT for the optimistic scenario (aka 25 ns) in CSA14.

  • POSTLS170_V6 GT
    • updated AlCa conditions for the 50 ns CSA14 scenario
    • technical fixes (a.k.a. migrations of the accounts in prod)
    • enabling Ecal full readout for pre-mixing
    • adding SiStrip offsets for APV cycle phase producers.
  • POSTLS170_V7 GT
    • enabling Ecal full readout for pre-mixing
    • adding SiStrip offsets for APV cycle phase producers.

Salvatore Di Guida added 2 commits March 14, 2014 11:52
…SiStrip offsets for APV cycle phase producers.

Signed-off-by: Salvatore Di Guida <salvatore.di.guida@cern.ch>
…l full readout for pre-mixing, and SiStrip offsets for APV cycle phase producers.

Signed-off-by: Salvatore Di Guida <salvatore.di.guida@cern.ch>
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @diguida (Salvatore Di Guida) for CMSSW_7_0_X.

Adding CSA14 GTs and enabling pre-mixing in autoCond.py

It involves the following packages:

Configuration/AlCa

@cmsbuild, @Degano, @diguida, @rcastello, @nclopezo can you please review it and eventually sign? Thanks.
@ghellwig this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@davidlange6
Copy link
Contributor

+1

On Mar 14, 2014, at 12:38 PM, cmsbuild notifications@github.com
wrote:

A new Pull Request was created by @diguida (Salvatore Di Guida) for CMSSW_7_0_X.

Adding CSA14 GTs and enabling pre-mixing in autoCond.py

It involves the following packages:

Configuration/AlCa

@cmsbuild, @Degano, @diguida, @rcastello, @nclopezo can you please review it and eventually sign? Thanks.
@ghellwig this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.


Reply to this email directly or view it on GitHub.

@diguida
Copy link
Contributor Author

diguida commented Mar 14, 2014

+1
@davidlange6 As specified in #2857 the GTs here remove Ecal Zero suppression using labels, so they are usable for pre mixing for both Run2 scenarios.
@franzoni these are the last and final GTs for CSA14: by adding these to the release, we can ease the validation, as we can seal up 7_0_1 for how it'll be used for CSA14. Please note that we added the APV offsets for compatibility, in order #2724 to work.

@cmsbuild
Copy link
Contributor

nclopezo added a commit that referenced this pull request Mar 14, 2014
Configuration/AlCa -- Adding CSA14 GTs and enabling pre-mixing in autoCond.py
@nclopezo nclopezo merged commit b16b6fd into cms-sw:CMSSW_7_0_X Mar 14, 2014
@cmsbuild
Copy link
Contributor

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

4 participants