<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:14pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<meta content="text/html; charset=UTF-8">
<div dir="ltr">Dear <span>Hċkan,</span>
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size: 14pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;">
<p><span><br>
</span></p>
<p><span>unfortunately, I am just now starting with the check and immediatly run into the compile problem from ~an hour ago.</span></p>
<p><span><br>
</span></p>
<p><span>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.</span></p>
<p><span><br>
</span></p>
<p><span><br>
</span></p>
<p><span><br>
</span></p>
<p><span><br>
</span></p>
<p><span>Best greetings</span></p>
<p><span>Günter<br>
</span></p>
<p><span><br>
</span></p>
<p><span><br>
</span></p>
<br>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>Von:</b> subexp-daq <subexp-daq-bounces@lists.chalmers.se> im Auftrag von Hċkan T Johansson <f96hajo@chalmers.se><br>
<b>Gesendet:</b> Montag, 15. Juli 2024 15:51:50<br>
<b>An:</b> Discuss use of Nurdlib, TRLO II, drasi and UCESB.<br>
<b>Betreff:</b> Re: [subexp-daq] NURDLIB - SIS3316 - major rework of check_hit</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt">
<div class="PlainText"><br>
Dear Günter,<br>
<br>
I guess the branch SIS3316_check_hit_reworked_rebased has now seen some <br>
use, such that it can be assumed to work and merged?<br>
<br>
Cheers,<br>
Hċkan<br>
<br>
<br>
<br>
On Fri, 14 Jun 2024, Hans Toshihide Törnqvist wrote:<br>
<br>
> Dear Günter,<br>
><br>
> I had a look, it seemed fine.<br>
><br>
> There is a rebased version against the current main:<br>
> SIS3316_check_hit_reworked_rebased<br>
><br>
> Due to some new safety checks, I had to remove a sis3316-only config <br>
> function and instead rely on common existing config functions, so please <br>
> try it out and see that it works ok. If this runs I will put all of it <br>
> in 'main'!<br>
><br>
> Best regards,<br>
> Hans<br>
><br>
><br>
> On 2024-06-10 18:54, Weber, Guenter Dr. wrote:<br>
>> Dear all,<br>
>> <br>
>> <br>
>> I just pushed a new version of the sis3316.c, where check_hit was <br>
>> rewritten. The reason was that the existing code did not work together <br>
>> with the discard option for the channel readout. Please note that in <br>
>> readout_dma I am now using an excess bit (not necessary to represent <br>
>> even the largest number of raw sample words possible) to store the <br>
>> information that an event was discarded. This is then used in check_event.<br>
>> <br>
>> <br>
>> As I did a lot of changes, please have an extra close look at the code.<br>
>> <br>
>> <br>
>> If everyone is happy with the new check_event, I will also make changes <br>
>> to the discard routine in readout_dma as at the moment, some assumptions <br>
>> on the settings of the module are hard coded - at least that is my <br>
>> impression at the moment.<br>
>> <br>
>> <br>
>> <br>
>> <br>
>> Thanks a lot!<br>
>> <br>
>> <br>
>> <br>
>> <br>
>> Best greetings<br>
>> <br>
>> Günter<br>
>> <br>
>> <br>
>> <br>
> -- <br>
> subexp-daq mailing list<br>
> subexp-daq@lists.chalmers.se<br>
> <a href="https://lists.chalmers.se/mailman/listinfo/subexp-daq">https://lists.chalmers.se/mailman/listinfo/subexp-daq</a><br>
></div>
</span></font></div>
</body>
</html>