[subexp-daq] NURDLIB - SIS3316 - major rework of check_hit

[Weber, Guenter Dr.]

Dear Håkan,


unfortunately, I am just now starting with the check and immediatly run into the compile problem from ~an hour ago.


I will be on travel for the next two days, but if the DAQ is running again, we can do the checks within a day or two.





Best greetings

Günter



________________________________
Von: subexp-daq <subexp-daq-bounces at lists.chalmers.se> im Auftrag von Håkan T Johansson <f96hajo at chalmers.se>
Gesendet: Montag, 15. Juli 2024 15:51:50
An: Discuss use of Nurdlib, TRLO II, drasi and UCESB.
Betreff: Re: [subexp-daq] NURDLIB - SIS3316 - major rework of check_hit


Dear Günter,

I guess the branch SIS3316_check_hit_reworked_rebased has now seen some
use, such that it can be assumed to work and merged?

Cheers,
Håkan



On Fri, 14 Jun 2024, Hans Toshihide Törnqvist wrote:

> Dear Günter,
>
> I had a look, it seemed fine.
>
> There is a rebased version against the current main:
> SIS3316_check_hit_reworked_rebased
>
> Due to some new safety checks, I had to remove a sis3316-only config
> function and instead rely on common existing config functions, so please
> try it out and see that it works ok. If this runs I will put all of it
> in 'main'!
>
> Best regards,
> Hans
>
>
> On 2024-06-10 18:54, Weber, Guenter Dr. wrote:
>> Dear all,
>>
>>
>> I just pushed a new version of the sis3316.c, where check_hit was
>> rewritten. The reason was that the existing code did not work together
>> with the discard option for the channel readout. Please note that in
>> readout_dma I am now using an excess bit (not necessary to represent
>> even the largest number of raw sample words possible) to store the
>> information that an event was discarded. This is then used in check_event.
>>
>>
>> As I did a lot of changes, please have an extra close look at the code.
>>
>>
>> If everyone is happy with the new check_event, I will also make changes
>> to the discard routine in readout_dma as at the moment, some assumptions
>> on the settings of the module are hard coded - at least that is my
>> impression at the moment.
>>
>>
>>
>>
>> Thanks a lot!
>>
>>
>>
>>
>> Best greetings
>>
>> Günter
>>
>>
>>
> --
> subexp-daq mailing list
> subexp-daq at lists.chalmers.se
> https://lists.chalmers.se/mailman/listinfo/subexp-daq
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.chalmers.se/pipermail/subexp-daq/attachments/20240715/cca7d984/attachment.html>