Feature #1919
Upgrade to ROOT6
0%
Description
I would like to upgrade MAUS to ROOT6 for the following reasons:
- maintain our ability to build MAUS on modern systems (ROOT5 and GCC6 don't seem to work together);
- enjoy the latest updates to ROOT which only go in ROOT6;
- generally keep the MAUS code base current.
When I presented this at an analysis meeting a few concern were raised:
1. Will ROOT6 MAUS be able to open our existing processed data?
2. Will it alter any physics results?
3. Will it mess up onrec?
4. Will it work my personal scripts?
I have addressed these questions (at least in part):
1. Yes, checked and no problem
2. I ran an MC of run 8681, 1000 events, and compared results, they were identical (output files attached)
3. I need the onrec code to say for sure, but the test plots look fine, plots in general look identical
4. There may need to be some small tweaks, principally changing . to -> to access pointers
NB: ROOT6 MAUS use slightly more memory than ROOT5, but otherwise the performance seems to be identical
Please let me have any more comments or concerns here.
Files
Related issues
Updated by Dobbs, Adam over 6 years ago
- File maus_resource_usage_release.eps maus_resource_usage_release.eps added
- File maus_resource_usage_merge.eps maus_resource_usage_merge.eps added
- File maus_resource_usage_root6.eps maus_resource_usage_root6.eps added
Attached are some plots for memory usage in release 2.8.3, the merge branch (root5) and my root6 branch. Memory usage is up, but remains acceptable.
Updated by Dobbs, Adam over 6 years ago
- File 08681_onrec_MAUSv2-8-3.tar.gz 08681_onrec_MAUSv2-8-3.tar.gz added
- File 08681_onrec_root6.tar.gz 08681_onrec_root6.tar.gz added
I have now analysed the output of onrec using ROOT6, and compared with vanilla release MAUS-v2.8.3. The results are attached. The plots appear identical with 2 exceptions:
- ROOT6 shows an increased memory usage, as expected (no problem);
- the TOF spacepoints vs spill number plots are not populating correctly under ROOT6.
The second point will need addressing in some way before switching to ROOT6. The rest of the results are very encouraging however.
Updated by Dobbs, Adam over 6 years ago
Fixed the TOF spacepoint vs. spill number issue. Require a small modification to src/reduce/ReduceCppTOFPlot/ReduceCppTOFPlot.cc to allow histogram rebinning:
_htof0_nspVspill->SetCanExtend(TH1::kXaxis);
for each tof detector.
Onrec is now good under ROOT6.