Skip to content

Nsj BadHit class#1094

Merged
aaust merged 12 commits intomasterfrom
nsj_badhit
Feb 10, 2026
Merged

Nsj BadHit class#1094
aaust merged 12 commits intomasterfrom
nsj_badhit

Conversation

@nsjarvis
Copy link
Copy Markdown
Contributor

@nsjarvis nsjarvis commented Feb 6, 2026

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.

r133245

@gluex
Copy link
Copy Markdown

gluex commented Feb 6, 2026

Comment thread src/plugins/Utilities/SConscript Outdated
@gluex
Copy link
Copy Markdown

gluex commented Feb 6, 2026

Build status for this pull request: FAILURE

Build log: /work/halld/pull_request_test/halld_recon^nsj_badhit/make_nsj_badhit.log
Build report: /work/halld/pull_request_test/halld_recon^nsj_badhit/report_nsj_badhit.txt
Location of build: /work/halld/pull_request_test/halld_recon^nsj_badhit

Will move this into a separate PR
This reverts commit d49937b.
@nsjarvis
Copy link
Copy Markdown
Contributor Author

nsjarvis commented Feb 6, 2026

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
Utilities/merge_rawevents
monitoring/DAQ_online
monitoring/EVNT_online
monitoring/pedestal_online

and also listed in libraries/CMakeLists.txt   

@gluex
Copy link
Copy Markdown

gluex commented Feb 6, 2026

@nsjarvis
Copy link
Copy Markdown
Contributor Author

Examples:
hd_rawdata_133245_000.evio (DIRC/SSP)
hd_rawdata_101622_001.evio (fa250)
I haven't found an example for the helicity board.

I have not tried to fix any event decoding problems. I might try that in a separate PR.

@gluex
Copy link
Copy Markdown

gluex commented Feb 10, 2026

Copy link
Copy Markdown
Contributor

@aaust aaust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great addition to the monitoring tools

@aaust aaust merged commit 09d3b99 into master Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants