search

jpata / tthbb13

2 stars 5 forks source link

Add fully hadronic in ttH code #133

Closed silviodonato closed 8 years ago

silviodonato commented 8 years ago

I tried to keep the two parts (leptonic and hadronic) independently just to avoid conflicts in future. In the common conflicting code, I left the leptonic part but I also added the hadronic code as comment.

jpata commented 8 years ago

looks mostly fine, I would give others a chance to have a look as well. I would wherever possible make FH/leptonic a flag and not a commentable option.

silviodonato commented 8 years ago

I think that the only thing is missing is a flag to separate the FH and leptonic work-flow. How do you suggest to do it? Environment variable? External config file?

d-salerno commented 8 years ago

It looks ok to me.

silviodonato commented 8 years ago

@d-salerno I've added fh_j8_t3 and fh_j9_t4 in: MEAnalysis/python/MEAnalysis_cfg_heppy.py MEAnalysis/python/inputs.py Plotting/python/Datacards/config_fh.cfg Plotting/python/joosep/controlPlot.py

jpata commented 8 years ago

looks nice, fine to merge by me

silviodonato commented 8 years ago

If @gkasieczka agrees, I think you can merge it.