JeffersonLab / sim-recon

Simulation and Reconstruction for GlueX
9 stars 14 forks source link

prevent a crash in TAGM_online if nsamples_pedestal is 0 #988

Closed aaust closed 6 years ago

aaust commented 6 years ago

default error message provided by JANA

sdobbs commented 6 years ago

Yikes, do we have data with this condition? Which run?

aaust commented 6 years ago

a few, e.g. 040348

sdobbs commented 6 years ago

Thanks, I hope this is due to the non-standard DAQ...

On Thu, Nov 30, 2017 at 11:04 AM Alex Austregesilo notifications@github.com wrote:

a few, e.g. 040348

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/JeffersonLab/sim-recon/pull/988#issuecomment-348234002, or mute the thread https://github.com/notifications/unsubscribe-auth/ABIJaiCqlIWOIRNe-SlKC-6_d6zJ70flks5s7tIBgaJpZM4QwyUr .

gluex commented 6 years ago

Test status for this pull request: SUCCESS

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

Build log: /work/halld/pull_request_test/sim-recon^aaust_TAGM_online_div0/make_aaust_TAGM_online_div0.log Build report: /work/halld/pull_request_test/sim-recon^aaust_TAGM_online_div0/report_aaust_TAGM_online_div0.txt Location of build: /work/halld/pull_request_test/sim-recon^aaust_TAGM_online_div0