Skip to content

Add detelement to map struct and fill it for NP02 maps#79

Merged
aolivier23 merged 1 commit intodevelopfrom
trj_addcrpnum_march19_2025
Mar 21, 2025
Merged

Add detelement to map struct and fill it for NP02 maps#79
aolivier23 merged 1 commit intodevelopfrom
trj_addcrpnum_march19_2025

Conversation

@tomjunk
Copy link
Copy Markdown
Member

@tomjunk tomjunk commented Mar 19, 2025

On Alessandro's request, I added a column to the ProtoDUNE-VD map structure called detelement, and filled it with the CRP number. This map structure is meant to be general enough to be used for horizontal drift (not being used yet so as not to disrupt workflows), so that's why it is called detelement -- it is to be filled with the APA number in horizontal-drift geometries. It was previously removed because the APA number coincided with the crate number, but that is not true for top drift electronics in the vertical drift detectors.

I tested it on np04hd_raw_run028550_0000_dataflow5_datawriter_0_20240805T100707.hdf5 and got identical-looking raw event displays.

@tomjunk tomjunk requested a review from vgalymov March 19, 2025 21:12
Copy link
Copy Markdown
Contributor

@vgalymov vgalymov left a comment

Choose a reason for hiding this comment

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

Ok, the map is identical to the previous one apart from the added detelement ID.

@aolivier23
Copy link
Copy Markdown
Contributor

trigger build

@FNALbuild
Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

@FNALbuild
Copy link
Copy Markdown
Collaborator

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

@FNALbuild
Copy link
Copy Markdown
Collaborator

⚠️ CI build for DUNE Succeeded with warning at phase build on slf7 for c14:prof - ignored warnings for build -- details available through the CI dashboard

parent CI build details are available through the CI dashboard

@FNALbuild
Copy link
Copy Markdown
Collaborator

⚠️ CI build for DUNE Warning at phase ci_tests DUNE on slf7 for e26:prof - ignored warnings for build -- details available through the CI dashboard

🚨 For more details about the warning phase, check the ci_tests DUNE phase logs

parent CI build details are available through the CI dashboard

@aolivier23 aolivier23 merged commit 1c888b7 into develop Mar 21, 2025
4 checks passed
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.

4 participants