-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hodoscope Data Reconstruction Tasks #448
Comments
The classes 'org.hps.readout.hodoscope.HodoscopeReadoutRawConverter' and 'org.hps.readout.hodoscope.HodoscopeRawConverterReadoutDriver' have been written to provide a working raw converter for readout in the iss420 branch. These can likely be modified with relatively little effort to support the conversion of readout ADC hodoscope hits into energy hits in recon as well. Presently, the driver is very highly reduced from the original recon variant, but it will probably only need the Mode-1 handling code re-added. If recon-level energy and time calibrations are needed, it would be easy to extend the driver and add that functionality there. The important thing will be to determine what functionality we will want for the hodoscope raw converter at the recon level. The options from the 2016 calorimeter raw converter include:
If few or none of these are needed, this may be quite simple. If most are needed, it will require more work. |
Thanks, Kyle. We can pull some files from iss421 onto the recon branch if
needed. I don't think we're quite there yet though...
…On Wed, May 1, 2019 at 10:58 AM Kyle McCarty ***@***.***> wrote:
The classes org.hps.readout.hodoscope.HodoscopeReadoutRawConverter and
org.hps.readout.hodoscope.HodoscopeRawConverterReadoutDriver have been
written to provide a working raw converter for readout in the iss420
branch. These can likely be modified with relatively little effort to
support the conversion of readout ADC hodoscope hits into energy hits in
recon as well.
Presently, the driver is very highly reduced from the original recon
variant, but it will probably only need the Mode-1 handling code re-added.
If recon-level energy and time calibrations are needed, it would be easy to
extend the driver and add that functionality there.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#448 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAQXV4YDZWT3UVLZYJ4OABTPTEBNTANCNFSM4HGQXHKQ>
.
|
So, I just ended up updating the raw converters even more. There is a new package called
I have made several implementations of these drivers as well. For readout, I have created I have also created I still need to check exactly what is different between Mode-3 in readout and recon before implementing any recon Mode-3 drivers. They may be the same thing. I also took the opportunity to purge the 2014 legacy support. No one actually uses this. I additionally removed the constant gain option, as it is only used in a single monitoring driver and probably really should be deprecated out. This can be added back in if really needed without much difficulty, though. Other things that are missing include some support Mode-1 to Mode-7 conversion. Most of the code is there, but it looks like no one ever finished it. We also need to complete the controller drivers and perform some serious testing. If it all works out, though, I think that we can use this much cleaner implementation of the raw converters to replace the rather over-bloated and ambiguous Note: This is all on iss420 still, since I'm using the changes to support the hodoscope readout chain as well. |
I am planning to work on the reconstruction, and on the list that Kyle wrote. For this I am planning to create a new branch iss448 |
See issue #490 where this task is continuing. |
Conversion of Hodoscope raw data to CalorimeterHit objects suitable for reconstruction.
TODO: Create detailed task list here.
The text was updated successfully, but these errors were encountered: