search

JeffersonLab / sim-recon

Simulation and Reconstruction for GlueX
9 stars 14 forks source link

fix logic in useWeight boolean #1133

Closed mashephe closed 6 years ago

mashephe commented 6 years ago

fix what appears to be a long-standing bug in feeding weighted events into AmpTools via the GlueX ROOTDataReader

gluex commented 6 years ago

Test status for this pull request: SUCCESS

Summary: /work/halld/pull_request_test/sim-recon^rootreader_weight_fix/tests/summary.txt Logs: /work/halld/pull_request_test/sim-recon^rootreader_weight_fix/tests/log

Build log: /work/halld/pull_request_test/sim-recon^rootreader_weight_fix/make_rootreader_weight_fix.log Build report: /work/halld/pull_request_test/sim-recon^rootreader_weight_fix/report_rootreader_weight_fix.txt Location of build: /work/halld/pull_request_test/sim-recon^rootreader_weight_fix

mikestaib commented 6 years ago

Sorry, thought I fixed this a year ago...but alas, only in the reader I was using with weights...

Enjoy the work fest!

ROOTDataReaderWithTCut::ROOTDataReaderWithTCut( const vector< string >& args ): UserDataReader< ROOTDataReaderWithTCut >( args ), m_eventCounter( 0 ), m_useWeight( true )

gluex commented 6 years ago

Test status for this pull request: SUCCESS

Summary: /work/halld/pull_request_test/sim-recon^rootreader_weight_fix/tests/summary.txt Logs: /work/halld/pull_request_test/sim-recon^rootreader_weight_fix/tests/log

Build log: /work/halld/pull_request_test/sim-recon^rootreader_weight_fix/make_rootreader_weight_fix.log Build report: /work/halld/pull_request_test/sim-recon^rootreader_weight_fix/report_rootreader_weight_fix.txt Location of build: /work/halld/pull_request_test/sim-recon^rootreader_weight_fix