cms-analysis / flashgg

19 stars 159 forks source link

Trigger sf ul18 #1269

Closed rchatter closed 3 years ago

rchatter commented 3 years ago

This PR is created on behalf of @Soumyatifr to add the Trigger UL18 efficiency SFs.

youyingli commented 3 years ago

Hi @rchatter, there are many dummy commits and conflicts which are hard to review. Maybe you can try to re-PR with only a commit that is trigger sf UL18. You can even send the trigger SF to me and I can make PR including vertexID UL18. Sorry for the inconvenience.

rchatter commented 3 years ago

Sorry for this I had installed the master afresh and just modified on file. Something diverged in my fork. I am closing this PR and sending you the numbers.