[Halld-tracking-hw] New f125 data format in sim-recon

[Naomi Jarvis]

Thank you.  I didn’t attempt the FDC formats because I had no idea where to put the peak time.

I wondered if we could have a Df125CDC which would contain pedestal, time, timeqf, amplitude, integral, the whole lot, instead of all the different Df125Pulse things, Pedestal, Peak, Integral etc.  It wouldn’t make any difference to DCDChits level or higher but it would make the lower level code less cumbersome.  

Naomi.



On Nov 9, 2015, at 12:04 PM, David Lawrence <davidl at jlab.org> wrote:

> Hi All,
> 
> I’ve just pushed a new branch onto github with modifications to support the new f125 data
> formats. I modified Naomi’s code a little for the CDC format (data type 5) and then added
> cases for the FDC formats (data types 6 and 9).
> 
> As part of this, I added a new “nsamples” member to the Df125PulsePedestal class. 
> This is not only more natural, but means we don’t need to create a Df125PulseIntegral
> object when when taking data of FDC type 9. I’ve also added a “peak_time” member to
> the Df125PulseTime class. This will be set to 0 when created for data types that don’t
> have this value.
> 
> I have not yet put in a pull request for this due partly to limited testing, but also because
> I’d like to get feedback from people if they have concerns with how this is being done.
> If you want to check out the revised code, it is in the branch:
> 
> davidl_newf125_formats
> 
> Regards,
> -David
> _______________________________________________
> Halld-tracking-hw mailing list
> Halld-tracking-hw at jlab.org
> https://mailman.jlab.org/mailman/listinfo/halld-tracking-hw

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.jlab.org/pipermail/halld-tracking-hw/attachments/20151109/4b1386ea/attachment.html>