-
Notifications
You must be signed in to change notification settings - Fork 53
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Added ensemble support for RingCounting tool. (#278)
* add RingCounting tool * RingCounting.py: added model loading and documentation * RingCounting.py: fix bug * RingCounting.py: rm unimplemented method call * RingCounting.py: fixed pmt masking, fixed file loading, added files_to_load path line commenting support * RingCounting.py: + docStrings + saving controlled by [[save_to]] * added RingCountingConfig, fixed loading error in RingCountingTool * RingCounting.py: + logging of successful file loading + pep8 formatting * Filled RingCountingTool README.md * Created RingCounting Tool (#1) Created RingCounting Tool --------- Co-authored-by: Daniel T. Schmid <[email protected]> * Update files_to_load.txt noop to trigger CI test * Fixed mrdtotalentries being smaller than tanktotalentries throwing a fatal error in LoadRawData.cpp when build type is set to "MRD", since in this case only MRD data is expected to be loaded. * Improved clarity of the load and save operation of the RingCounting tool: - Renamed load_from_file -> load_from_csv - Introduced variable save_to_csv - Updated documentation at beginning of RingCounting.py to reflect changes - Updated README.md to reflect changes - Updated UserTools/RingCounting/RingCountingConfig to reflect changes Users using the RingCounting tool need to update their RingCountingConfig files following this change. * Added ensemble support for RingCounting tool. - Moved logic for processing predictions to method process_predictions - Added new configuration variables model_is_ensemble, ensemble_model_count,ensemble_prediction_combination_mode - Added support for loading ensemble models and processing predictions - Added support for average and majority-voting ensemble - Added storing of majority-voting ensemble class prediction in new variables RingCountingVoting{SR,MR}Prediction in RecoEvent BoostStore - Updated documentation within RingCounting.py - Updated README.md - Updated configfiles/RingCounting/RingCountingConfig Users using the RingCounting tool need to update their RingCountingConfig file(s) following this change. --------- Co-authored-by: Daniel T. Schmid <[email protected]> Co-authored-by: marc1uk <[email protected]>
- Loading branch information
1 parent
db4358b
commit 0d9aca0
Showing
3 changed files
with
123 additions
and
18 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters