Skip to content

New DCRB firmware#472

Closed
raffaelladevita wants to merge 10 commits intodevelopmentfrom
iss223-dcrbWidth
Closed

New DCRB firmware#472
raffaelladevita wants to merge 10 commits intodevelopmentfrom
iss223-dcrbWidth

Conversation

@raffaelladevita
Copy link
Collaborator

  • added decoding of evio banks produced by the new DCRB firmware, saving time-over-threshold to DC::tdc.width;
  • added TDC hits filtering based value, tot, and multiplicity

@raffaelladevita
Copy link
Collaborator Author

Tested on run 20287:

Keeping first hit in time in full range
Plot_02-13-2025_04 10 27_PM

@raffaelladevita
Copy link
Collaborator Author

raffaelladevita commented Feb 19, 2025

Performed more tests on run 20283 (70 nA) and compared to production run at the same current.

The plot below shows for regions 1, 2, and 3, from top to bottom:

  • the number of hits per channel group,
  • the TDC distribution,
  • the TOT distribution,
  • the time distance between the hit leading edge and the previous hit trailing edge,
  • the hit type where 0 is assigned to the first hit on a wire with TOT>50 and TDC within the range used in reconstruction, 100 to hits outside that range, 110 to hits below the TOT threshold, and 120 to the hits that pass the cuts but are not the first.
    Colors correspond to all hits in black, the first hit in red, the subsequent hits when the first hit has TOT<50 in green. The latter histogram is scaled by x10.
    Plot_02-19-2025_05 31 39_PM

The comparison between run 20283 and 20288 was done processing 10 evio files with denoising and AI-assisted tracking. The plots below show various distributions for positive tracks selected in sectors 2 and 5 that had the new DCRB firmware in run 20283. Plots for run 20283 and 20288 are shown in red and black, respectively. For a fair comparison, the histograms are scaled to the same number of electrons detected in sectors 1,3,4, and 6.
Plot_02-19-2025_05 36 05_PM

(RED=NEW, BLACK=OLD)

The relevant statistics are reported in the table below.

+--------------------------------------------------------------------------------------+
|     charge |      n. sl |   run 20288 |      run 20283|      gain |  
+--------------------------------------------------------------------------------------+
|        pos |        any |        97664 |        97825 |    1.0016 |      
|        pos |          6 |        88856 |        90580 |    1.0194 |      
|        pos |          5 |         8673 |         7125 |    0.8215 |     
+--------------------------------------------------------------------------------------+
+--------------------------------------------------------------------------------------+
|         type |            e |          eh+ |          eh- |      eh+/e |       eh-/e |
+--------------------------------------------------------------------------------------+
|    run 20288 |        45517 |        21375 |         5296 |     0.4696 |      0.1164 |
|    run 20283 |        45518 |        21510 |         5370 |     0.4726 |      0.1180 |
+--------------------------------------------------------------------------------------+

@baltzell baltzell linked an issue Feb 20, 2025 that may be closed by this pull request
@raffaelladevita
Copy link
Collaborator Author

These are the channel group multiplicity distributions as a function of the slot and group from run 20283
dcrbmult

@whit2333 whit2333 marked this pull request as ready for review March 25, 2025 12:43
@whit2333
Copy link
Collaborator

whit2333 commented Mar 25, 2025

Let's go ahead and merge this asap. I have some updates to mon12 which depend on it.
Here is a plot of with and w/o a 20ns ToT cut:
https://logbooks.jlab.org/entry/4343562

@whit2333 whit2333 requested a review from baltzell March 25, 2025 12:43
@c-dilks
Copy link
Member

c-dilks commented Mar 26, 2025

pre-commit.ci job will pass once #505 is merged; sorry we may have done the pre-commit activation steps a bit out of order

@whit2333
Copy link
Collaborator

@c-dilks we need this merged as soon as @raffaelladevita is ready.

@raffaelladevita
Copy link
Collaborator Author

Superseded by #509

@github-project-automation github-project-automation bot moved this from In Progress to Done in ALERT Software Roadmap Mar 27, 2025
@baltzell baltzell removed a link to an issue Apr 5, 2025
@baltzell baltzell deleted the iss223-dcrbWidth branch May 21, 2025 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants