search

jpata / tthbb13

2 stars 5 forks source link

FH updates: gc configs, switch in ME_heppy, multicrab, plotting scripts #67

Closed d-salerno closed 8 years ago

jpata commented 8 years ago

Looks good Daniel, just one thing: do you really want to copy

MEAnalysis/gc/projectSkim_FH.sh
MEAnalysis/gc/count_FH.sh

In this case we'll need to maintain two versions of these things. I would keep the scripts as common as possible. What exactly would you need to change in those scripts?

d-salerno commented 8 years ago

Hmm, the only reason for those scripts is to keep a different output directory, that I don't want to have to remember to change each time I update the repo. Perhaps we could compromise and make the output directory somewhere inside the CMSSW directory. Maybe {CMSSW_DIR}/src/TTH/Output/count (and skim)?

jpata commented 8 years ago

If you remove the copying of the files to _FH, I can merge this. PR #68 takes care of the output dir stuff.

jpata commented 8 years ago

Regarding MEAnalysis/gc/confs/QCD*.conf, if they have pretty similar runtimes per file, you could as well merge the configurations and specify the datasets like this:

dataset =
    /path/to/qcd1.dat
    /path/to/qcd2.dat

see https://github.com/jpata/tthbb13/blob/master/MEAnalysis/gc/confs/projectSkim.conf#L16