-
Notifications
You must be signed in to change notification settings - Fork 18
TODOs
Dennis Roy edited this page Jul 16, 2018
·
21 revisions
These are issues, problems and ideas, that concern the complete group (as they this not just appear an a very special use-case) and thus require discussions in the group to converge on a solution, that the complete group supports. The implementation of a solution should be done with care an will take a bit longer.
- can we remove datasetHelper? Only datasetHelperTwopz is used. I also don’t see it in Artus and Htt where in the later datasetsHelper2015 is used (only isData). datasetsHelper2015 can be probably removed too.
- fix the oppended issue on dublicating functionality
- Artus configurations via Python code instead of JSON files. Inspiration: https://github.com/KIT-CMS/KITHiggsToTauTau/tree/master/python/data/ArtusConfigs
- Remove duplicated quantities from Artus output ntuples. Prefer LV quantities over float components
This is a list of known issues in our framework, things that could be improved at some time, but that are not crucial. It also contains points that should be fixed during the usual analysis work as kind of personal issues.
- find a way to fasten scram b, it checks the whole repository after 1 change
- Add more documentation to important scripts in our framework.
- Fill the tutorial page about the combine tool with links and a small tutorial how combine is used in general.
- properly use getValidVertexIndex in KVertexSummary producer
- a filelist creation could use a call of a script that checks that the created file-list is not empty
- add the correct files to the cernbox and handles to define which of them to use in the tests. Especially the 2017 files.
- Add the Artus test to travis. Requires discussion on how we want it. Seems like Raphael already is doing some steps towards this
- compareRootFiles.py in Htt with option -a creates an error message if the number of leafs is different. It should be possible to compare the branches that match
-
GenParticleDecayTree
class: methodsDetermineDecayTree
needs to be corrected, because rho and a1 decays are identified as pi decays (To keep in mind: use ofstatus
and not onlypdgId
variable. Look also at theBuildDecayTree
inGenTauDecayProducer
). - Gen/RecoTauCPProducer: some refactoring is needed (e.g. if statements, naming convention, remove not needed functions)
- make the creation of these plots possible
- the default root plot module only supports one subplot canvas and Thomas is not sure, whether we should add more canvases here (because this duplicates many settings). He would propose to start with a new plotting modules to be run after the current one
- fix the automatic width adjustion for the option --function-display-result in FunctionPlot module. Example:
higgsplot.py -f "demo/once_tree" \
-x v_v0_Ks_inv_m_pi \
--live -i out_mc_full_finer_binning_new.root \
--nicks mc_nick -m LINE \
--x-label "Inv. mass of K daughters, GeV" --y-label "norm. entries" \
--analysis-modules NormalizeToUnity FunctionPlot \
--functions "[0]*(exp(-0.5*((x-[1])/[2])**2)) + [3]*x+[4]" \
--function-parameters 0.06,0.49769,0.003475,0.017,0.0051458 \
--function-ranges 0.45,0.55 --x-lims 0.4 0.6 \
--function-fit mc_nick \
--log-level debug \
--x-bins 1000,0.4,0.6 \
--function-display-result \
--function-fit-parameter-names "a" "b" "c" "d" "e"
- Fix the ignoring of bin susing log-scale for 3D histograms. Example:
higgsplot.py -f "demo/kaon_tree" \
-z v0_Ks_inv_m_pi -x hps_pt_1 -y hps_pt_2 \
-w "v0_Ks_inv_m_pi>0 && hps_pt_1>0" \
--live -i out_mc_full_matched_byReference.root \
--nicks mc_v0_Ks1 \
--z-bins 400,0.4,0.6 --x-bins "0 3 8 1000" --y-bins "0 3 8 1000" --z-lims 0.4 0.6 --x-lims 0.1 10 --y-lims 0.1 10 --x-log --y-log \
-m TEXTCOLZ \
--filename projection \
--analysis-modules NormalizeToUnity ProjectByFit \
--projection-functions "[0]*(exp(-0.5*((x-[1])/[2])**2)) + [3]*x+[4]" \
--projection-parameters 0.00615232,0.497674,0.00434818,0.00172523,0.00009 \
--projection-to-plot 1 \
--projection-to-nick mc_v0_Ks1 \
--projection-fit-range 0.4,0.6 \
--projection-fit-slices-z