Displaying report 1-1 of 1.
Reports until 11:24, Monday 07 October 2013
H1 ISC
jeffrey.kissel@LIGO.ORG - posted 11:24, Monday 07 October 2013 (8021)
One last change to h1ascimc model
J. Kissel, S. Ballmer, C. Wipf

After discussions with both Chris and Stefan, I have re-arranged how the IMC WFS' master switch, gain, and trigger bit are combined such that the implementation is a little less obfuscated and well-annotated in the Simulink model. I attach a new screenshot. In this new arrangement, each of the three variables -- all nominally 0 (zero) or 1 (one) for OFF and ON, respectively -- are simply multiplied together by one block, as opposed to feeding the trigger into a switch block that compares its value against the master switch (nominally 1 [one]) and a ground (equivalent 0 [zero]) [See identically named screenshot in LHO aLOG 7984 for previous implementation]. 

For historical purposes: Stefan had originally implemented the triggering with the comparator switch for philosophical reasons in that (at least with conventional c-code) it is bad practice to assume a trigger could only be 0 or 1 -- typically when programming in c, a flag is either zero or non-zero, where non-zero can be anything and should never be assumed to be one. However, the trigger library part used (from ${userapps}/release/isc/common/models/LSC_TRIGGER.mdl) is written to ONLY spit out a 0 or 1. Therefore this bad-practice failure-mode dos not apply, and we are free to make the output bit generation more clear.

As of this entry, the 
${userapps}/release/asc/h1/models/h1ascimc.mdl
has been re-compiled, re-installed, re-started, and re-stored. Since the mode cleaner is dead for the time being due to venting of the corner volume (light from the PSL has been shuttered), I cannot *confirm* all is still functional, but I'm 99% confident all is well.
Images attached to this report
Displaying report 1-1 of 1.