Skip to content

ADD: Hardware threshold files from unit-level data - #208

Merged
fhagemann merged 2 commits into
cositools:develop/emfrom
fhagemann:resource/hardware-thresholds
Sep 15, 2026
Merged

fhagemann merged 2 commits into
cositools:develop/emfrom
fhagemann:resource/hardware-thresholds

Conversation

@fhagemann

@fhagemann fhagemann commented Sep 15, 2026 •

Copy link
Copy Markdown

Needed by #190, addressing issue #195:

Based on the threshold finding app, @JarredMRoberts determined the hardware threshold files and uploaded them to Google Drive: https://drive.google.com/drive/folders/1FN4fp0_imv-opuJHoKn1N3DPwUUPe38A?usp=sharing

In this PR, we're adding the files for the individual flight detectors to resource/calibration/COSISMEX/unit, and a combined file containing all 16 flight detectors to resource/calibration/COSISMEX/payload.

I made some changes compared to the files that were on Google Drive:

  • renamed the files to follow the naming convention used for the simulated depth splines
  • added a first column STRIP_ID determined from the strip map to follow the format of resource/dee/dummy_hardware_thresholds.csv (which follows the format also of the average fast/slow threshold files that we determine in unit-level testing)

I'm opening as a draft for now, because I haven't tried running the code in #190 using these threshold files.
Will mark this as "ready for review" once I have tested this.

@fhagemann fhagemann linked an issue Sep 15, 2026 that may be closed by this pull request
@fhagemann

Copy link
Copy Markdown
Author

So, I ran nuclearizer using a cosima simulation for 542-1 on the STTC, and using the 542-1 hardware threshold file.
The energy histogram when applying the energy calibration looks reasonable (in the sense that it seems to be adding a noise peak):
image

Will test this more thoroughly tomorrow, but this looks promising (at least for 542-1)

@fhagemann
fhagemann marked this pull request as ready for review September 15, 2026 06:22
@fhagemann

Copy link
Copy Markdown
Author

Browsing through different strips (I did this for 542-1) seems to yield reasonable results in the DEE:
Hardware_thresholds

@ckierans ckierans left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, @fhagemann !

@fhagemann fhagemann changed the title ADD: Hardware threshold files from unit-level data to resource/calibration/COSISMEX/unit ADD: Hardware threshold files from unit-level data Sep 15, 2026
@fhagemann
fhagemann merged commit 65058f6 into cositools:develop/em Sep 15, 2026
1 check passed
@fhagemann
fhagemann deleted the resource/hardware-thresholds branch September 15, 2026 18:14
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.

Determine hardware threshold in Nuclearizer App

3 participants