search

padsley / k600analyser

Code for the K600 analyser including plugin codes for silicon, clover and HAGAR data
1 stars 4 forks source link

Double hits in fronts #30

Closed padsley closed 9 years ago

padsley commented 9 years ago

RN has noticed an error. From his e-mail:

Hi

When I have events with 2 hits in the MMM's I should have a vector with 2 entries for each of the datamembers of the SiliconData class, right?

As I understand it I should then get different plots if for example I plot DATA->Draw("SiliconEnergy[0]>>h(10000,0,10000)","SiliconHits==2","") compared to DATA->Draw("SiliconEnergy[1]>>h(10000,0,10000)","SiliconHits==2","")

However, I get the exact same thing in both cases. What really confuses me is that for the vector ADCValueBack I do indeed get 2 different spectra. So in principle this works. But for most of the other vectors the elements of the vector are 100% the same.

What am I not understanding here?

padsley commented 9 years ago

I think that this issue is due to entries where there is a double hit in the front TDCs which causes a front entry to be made twice. RN is checking this. We'll see if that's the issue. If it is, we'll look at remedies. If not, we'll keep trying to locate the problem.

padsley commented 9 years ago

While I think about it, the sort code currently is not able to cope (plus ca change) with TDC hits in fronts vs TDC hits in backs. Etc. There is only one TDC loop. So. That'll need to change at some point.

padsley commented 9 years ago

I'm not expecting you to fix this, Kevin... I'm just adding you as the assignee so that I'm reminded that you're working on something related to it.

Though... if you fix the problem, do let me know. :P

padsley commented 9 years ago

Retief noted that this problem was due to the ntdc problems introducted earlier when I thought that ntdc was number of tdc hits not number of tdc words. This has now been fixed. I'm closing the issue.