Closed alicerobson closed 7 years ago
All tests succeeded.
OK, so I think this addresses the things we have discussed. Hopefully I have understood correctly.
As commented above, I ended up adding a variable into the configuration.py file. I am happy to change this if you have a better plan.
All tests succeeded.
All tests succeeded.
This pull request makes changes to allow FCCSW generated papas reconstructed particles to be imported and used within Heppy in place of carrying out the papas reconstruction within python.
Changes made include
The example using this approach is analysis_ee_Z_bb_FCCSW.py which mimics analysis_ee_Z_bb.py
Note:- The values of the test test_analysis_ee_Z_bb.py have been made rather less precise. First, this is needed because otherwise a change in the random seed makes the tests fail. Secondly, it is not entirely straightforward to make random number generation match between the python only approach and the FCCSW+python (except for the first event) so uncertainty due to random seed need to be allowed for. I spent a lot of time testing to come to this conclusion.