uwcms / FinalStateAnalysis

An analysis framework for the Compact Muon Solenoid
3 stars 43 forks source link

Higgs MC PatTuples #37

Closed dabelknap closed 12 years ago

dabelknap commented 12 years ago

Hi Tapas, I need patTuples made for the following samples in "/hdfs/store/mc/Summer12_DR53X"

GluGluToHToZZTo4L_M-115_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-116_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-117_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-118_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-119_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-120_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-121_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-122_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-123_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-124_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-125_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-126_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-127_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-128_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-129_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-130_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-135_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-140_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-145_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-150_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-160_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-170_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-180_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-190_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-200_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-220_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-250_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-275_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-300_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-325_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-350_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-375_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-400_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-425_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-450_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-475_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-500_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-525_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-550_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-575_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-600_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-650_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-700_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-750_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-800_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-850_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-900_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-950_8TeV-powheg-pythia6 GluGluToHToZZTo4L_M-1000_8TeV-powheg-pythia6

VBF_HToZZTo4L_M-115_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-116_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-117_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-118_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-119_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-120_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-121_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-122_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-123_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-124_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-125_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-126_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-127_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-128_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-129_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-130_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-135_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-140_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-145_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-150_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-160_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-170_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-180_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-190_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-200_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-220_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-250_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-275_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-300_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-325_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-350_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-375_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-400_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-425_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-450_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-475_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-500_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-525_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-550_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-575_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-600_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-650_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-700_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-750_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-800_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-850_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-900_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-950_8TeV-powheg-pythia6 VBF_HToZZTo4L_M-1000_8TeV-powheg-pythia6

tsarangi commented 12 years ago

Austin,

Are all these available at T2 ? Are the datadefs available in the FinalStateAnalysis? Give an example AOD dataset for each of the kind ! One for VBF, one for GluGlu.

ekfriis commented 12 years ago

Hi all,

I think to reduce Tapas' load (he is not an expert on our analyses) we should make an entry in the datadefs for samples we requesting:

So for this case, Austin, you need to edit:

https://github.com/uwcms/FinalStateAnalysis/blob/master/MetaData/python/data8TeVNew.py

in your forked repository, commit, push to github, and then send a pull request. Let me know if you need help with any of this.

Evan

On Wed, Oct 10, 2012 at 11:01 AM, Tapas Sarangi notifications@github.comwrote:

Austin,

Are all these available at T2 ? Are the datadefs available in the FinalStateAnalysis? Give an example AOD dataset for each of the kind ! One for VBF, one for GluGlu.

— Reply to this email directly or view it on GitHubhttps://github.com/uwcms/FinalStateAnalysis/issues/37#issuecomment-9294495.

dabelknap commented 12 years ago

I just submitted the pull request that have the definitions for the samples I need patTuplized. Tapas, once you have those changes, the sample names are

GluGluToHToZZTo4L_M-*_8TeV-powheg-pythia6
VBF_HToZZTo4L_M-*_8TeV-powheg-pythia6
ekfriis commented 12 years ago

Hi Austin,

See my comment on your pull request:

https://github.com/uwcms/FinalStateAnalysis/pull/39/files#L2R216

Do you want to really add the pat photons?

Evan

On Wed, Oct 10, 2012 at 1:33 PM, Austin Belknap notifications@github.comwrote:

I just submitted the pull request that have the definitions for the samples I need patTuplized. Tapas, once you have those changes, the sample names are

GluGluToHToZZTo4L_M-8TeV-powheg-pythia6 VBF_HToZZTo4L_M-8TeV-powheg-pythia6

— Reply to this email directly or view it on GitHubhttps://github.com/uwcms/FinalStateAnalysis/issues/37#issuecomment-9298104.

dabelknap commented 12 years ago

Hi Evan,

Were those still in there? Sorry about that. You can ignore all changes except the ones to data8TeVNew.py.

-Austin

On Oct 10, 2012, at 1:54 PM, Evan K. Friis wrote:

Hi Austin,

See my comment on your pull request:

https://github.com/uwcms/FinalStateAnalysis/pull/39/files#L2R216

Do you want to really add the pat photons?

Evan

On Wed, Oct 10, 2012 at 1:33 PM, Austin Belknap notifications@github.comwrote:

I just submitted the pull request that have the definitions for the samples I need patTuplized. Tapas, once you have those changes, the sample names are

GluGluToHToZZTo4L_M-8TeV-powheg-pythia6 VBF_HToZZTo4L_M-8TeV-powheg-pythia6

— Reply to this email directly or view it on GitHubhttps://github.com/uwcms/FinalStateAnalysis/issues/37#issuecomment-9298104.

— Reply to this email directly or view it on GitHub.

ekfriis commented 12 years ago

Hi Austin,

I don't think we can edit a pull request. I think what you need to do is undo that commit in your local area, push the change to your github, then click pull request again - this will amend the pull request undoing the changes.

If you look here: https://github.com/uwcms/FinalStateAnalysis/pull/39

you can see that the hash (ID) of the offending commit is 116ac4d.

Since you just want to undo this, the easiest way is to (in your area):

git revert 116ac4d

(this will drop you into an editor to give a commit message, you can use the default one)

and then push to your github, then submit a pull request again.

Evan

On Wed, Oct 10, 2012 at 1:56 PM, Austin Belknap notifications@github.comwrote:

Hi Evan,

Were those still in there? Sorry about that. You can ignore all changes except the ones to data8TeVNew.py.

-Austin

On Oct 10, 2012, at 1:54 PM, Evan K. Friis wrote:

Hi Austin,

See my comment on your pull request:

https://github.com/uwcms/FinalStateAnalysis/pull/39/files#L2R216

Do you want to really add the pat photons?

Evan

On Wed, Oct 10, 2012 at 1:33 PM, Austin Belknap < notifications@github.com>wrote:

I just submitted the pull request that have the definitions for the samples I need patTuplized. Tapas, once you have those changes, the sample names are

GluGluToHToZZTo4L_M-8TeV-powheg-pythia6 VBF_HToZZTo4L_M-8TeV-powheg-pythia6

— Reply to this email directly or view it on GitHub< https://github.com/uwcms/FinalStateAnalysis/issues/37#issuecomment-9298104>.

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHubhttps://github.com/uwcms/FinalStateAnalysis/issues/37#issuecomment-9298644.

dabelknap commented 12 years ago

Hi Evan,

It looks like the pull request automatically updated itself to include the reverted commit. It's just as well since it won't let me submit another one while the current is still open.

-Austin

On Oct 10, 2012, at 2:00 PM, Evan K. Friis wrote:

Hi Austin,

I don't think we can edit a pull request. I think what you need to do is undo that commit in your local area, push the change to your github, then click pull request again - this will amend the pull request undoing the changes.

If you look here: https://github.com/uwcms/FinalStateAnalysis/pull/39

you can see that the hash (ID) of the offending commit is 116ac4d.

Since you just want to undo this, the easiest way is to (in your area):

git revert 116ac4d

(this will drop you into an editor to give a commit message, you can use the default one)

and then push to your github, then submit a pull request again.

Evan

On Wed, Oct 10, 2012 at 1:56 PM, Austin Belknap notifications@github.comwrote:

Hi Evan,

Were those still in there? Sorry about that. You can ignore all changes except the ones to data8TeVNew.py.

-Austin

On Oct 10, 2012, at 1:54 PM, Evan K. Friis wrote:

Hi Austin,

See my comment on your pull request:

https://github.com/uwcms/FinalStateAnalysis/pull/39/files#L2R216

Do you want to really add the pat photons?

Evan

On Wed, Oct 10, 2012 at 1:33 PM, Austin Belknap < notifications@github.com>wrote:

I just submitted the pull request that have the definitions for the samples I need patTuplized. Tapas, once you have those changes, the sample names are

GluGluToHToZZTo4L_M-8TeV-powheg-pythia6 VBF_HToZZTo4L_M-8TeV-powheg-pythia6

— Reply to this email directly or view it on GitHub< https://github.com/uwcms/FinalStateAnalysis/issues/37#issuecomment-9298104>.

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHubhttps://github.com/uwcms/FinalStateAnalysis/issues/37#issuecomment-9298644.

— Reply to this email directly or view it on GitHub.

ekfriis commented 12 years ago

No problem - that's how it's supposed to go - pull request, feedback, add commits to pull request, merge. Tapas, it is in the HEAD now, so you can pull and submit whenever.

tsarangi commented 12 years ago

there was a typo in the changes made by Austin in the datadefs of data8TeVNew.py. Now it's fixed and pushed to the central repo.

tsarangi commented 12 years ago

a lot of these samples are not there in T2 ! Have anyone checked them carefully ?

dabelknap commented 12 years ago

I got the dataset names from DAS and I checked /hdfs/store/mc/Summer12_DR53X to make sure we had the samples. I also double-checked to make sure I was inputing the correct mass values. Did I miss something?

iross commented 12 years ago

https://github.com/uwcms/FinalStateAnalysis/blob/master/MetaData/python/data8TeVNew.py#L278

The dataset paths here are still pointing to the 52 samples.

iross commented 12 years ago

Actually, it's fine just a little above: https://github.com/uwcms/FinalStateAnalysis/blob/master/MetaData/python/data8TeVNew.py#L247

But I guess they're being overwritten when the datasets get read.

tsarangi commented 12 years ago

there were two instances of GluGluToHToZZ... in the datadefs and the latter one was for 52X samples, which was overwriting the 53X ones defined earlier in the data8TeVNew.py

tsarangi commented 12 years ago

Fixed and pushed to central repo.

tsarangi commented 12 years ago

Status : SUBMITTED

dabelknap commented 12 years ago

Thanks Tapas

On Oct 10, 2012, at 5:27 PM, Tapas Sarangi wrote:

Status : SUBMITTED

— Reply to this email directly or view it on GitHub.

tsarangi commented 12 years ago

Status : All Done. A very few failed and resubmitted. They should be available in a couple of hours.

Output location : /hdfs/store/user/tapas/2012-10-02-8TeV-53X-PatTuple_ShareFSFix/

Closing this issue.