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

Heppy 74X integration #9029

Merged
merged 68 commits into from May 20, 2015
Merged

Heppy 74X integration #9029

merged 68 commits into from May 20, 2015

Conversation

cbernet
Copy link
Contributor

@cbernet cbernet commented May 11, 2015

Latest version of the Heppy analysis software package, tested in 7_4_1_patch1.
This PR affects one package that does not belong to Heppy, EgammaAnalysis/ElectronTools.
These changes have also been requested by @dimarco here: #9006.

@gpetruc @arizzi

emanueledimarco and others added 30 commits April 3, 2015 10:04
* add the switch to use full5x5 cluster shapes
Heppy updates to support the new electron IDs. Thanks Emanuele!
…e the possibility to include conversion veto in the cut-based eleID
…onID(), and allow setting mu_tightId to anything allowed by muonID(); also, add POG_Global_OR_TMArbitrated muon id
…en iso both with deltaR 0.3 and 0.4; extending photon sietaieta sideband to 0.015 in EB
@monttj
Copy link
Contributor

monttj commented May 18, 2015

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

@cbernet
Copy link
Contributor Author

cbernet commented May 18, 2015

Hello,

Could you please tell me what needs to be discussed so that I can get prepared,
and also tell me when is the next ORP meeting?

Many thanks,

Colin

Le 18 mai 2015 à 02:26, cmsbuild notifications@github.com a écrit :

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar


Reply to this email directly or view it on GitHub.

@cbernet
Copy link
Contributor Author

cbernet commented May 20, 2015

Hello,

Could you please let me know what needs to be discussed and when?
If there is in fact nothing to discuss, could somebody please merge?

Thanks,

Colin

Le 18 mai 2015 à 09:13, Colin Bernet colin.bernet@cern.ch a écrit :

Hello,

Could you please tell me what needs to be discussed so that I can get prepared,
and also tell me when is the next ORP meeting?

Many thanks,

Colin

Le 18 mai 2015 à 02:26, cmsbuild notifications@github.com a écrit :

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar


Reply to this email directly or view it on GitHub.

@gpetruc
Copy link
Contributor

gpetruc commented May 20, 2015

Hi Colin,

It was discussed at the ORP yesterday, and this and the other analysis-only
pull requests will be merged soon,

Giovanni

On Wed, May 20, 2015 at 7:07 AM, Colin Bernet notifications@github.com
wrote:

Hello,

Could you please let me know what needs to be discussed and when?
If there is in fact nothing to discuss, could somebody please merge?

Thanks,

Colin

Le 18 mai 2015 à 09:13, Colin Bernet colin.bernet@cern.ch a écrit :

Hello,

Could you please tell me what needs to be discussed so that I can get
prepared,
and also tell me when is the next ORP meeting?

Many thanks,

Colin

Le 18 mai 2015 à 02:26, cmsbuild notifications@github.com a écrit :

This pull request is fully signed and it will be integrated in one of
the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull
request requires discussion in the ORP meeting before it's merged.
@davidlange6, @Degano, @smuzaffar


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub
#9029 (comment).

@davidlange6
Copy link
Contributor

+1

@cbernet
Copy link
Contributor Author

cbernet commented May 20, 2015

Thanks all!
Colin

Le 20 mai 2015 à 17:00, cmsbuild notifications@github.com a écrit :

Merged #9029.


Reply to this email directly or view it on GitHub.

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