Conversation
|
Test status for this pull request: SUCCESS Summary: /work/halld/pull_request_test/halld_recon^nsj_badhit/tests/summary.txt Build log: /work/halld/pull_request_test/halld_recon^nsj_badhit/make_nsj_badhit.log |
|
Build status for this pull request: FAILURE Build log: /work/halld/pull_request_test/halld_recon^nsj_badhit/make_nsj_badhit.log |
Will move this into a separate PR This reverts commit d49937b.
|
I reverted the cleanup of JEventSource_EVIO.* because it is still used in these old and obscure plugins. 😜 Mea Culpa, I should have done a grep -r instead of just recompiling. Utilities/evio_writer and also listed in libraries/CMakeLists.txt |
|
Test status for this pull request: SUCCESS Summary: /work/halld/pull_request_test/halld_recon^nsj_badhit/tests/summary.txt Build log: /work/halld/pull_request_test/halld_recon^nsj_badhit/make_nsj_badhit.log |
|
Examples: I have not tried to fix any event decoding problems. I might try that in a separate PR. |
|
Test status for this pull request: SUCCESS Summary: /work/halld/pull_request_test/halld_recon^nsj_badhit/tests/summary.txt Build log: /work/halld/pull_request_test/halld_recon^nsj_badhit/make_nsj_badhit.log |
aaust
left a comment
There was a problem hiding this comment.
Great addition to the monitoring tools
There are some known evio format problems in the data from the f250, f125 and dirc. This PR introduces a new class DBadHit to contain the roc and slot info when one of these errors is found. There is a plugin and a macro to make a plot of the roc and slot, if errors are found. When it creates a BadHit it writes a message to jerr.
For the f125 it only logs a bad hit if the number of peaks is 1, to suppress the apparent errors occurring when that number has been recorded erroneously (usually few-bit errors, it appears as 9, 17, 25), as the CDC and FDC settings require (and always required) data for only 1 peak per hit.
It deals nicely with issues like 703
It does not do anything for the data errors affecting the swap_bank function.
The intention is to use the plugin for online monitoring, so that problematic rocs can be rebooted when needed.
The picture is was generated using the bad_hits plugin with 133245 file 0.