Reports until 22:52, Wednesday 23 December 2015
H1 CDS
thomas.shaffer@LIGO.ORG - posted 22:52, Wednesday 23 December 2015 (24429)
Fixed bug in VerbalAlarms

Today VerbalAlarms crashed due to a ValueError, but luckily we didn't need Verbs today because of the environment.

The problem came from code I added on December 9 that would stop the tempurature alarms from repeating too often. I it would look through a date string for the 9th-11th elements, which should be the minutes, and then see if it was five minutes since the last alarm. This was good and all until the date changed from a single digit to two digits. That was something I should have seen, my fault. The reason it did not show up until now is because it was under a few conditionals that have not been met since the 9th.

The fix was two part. First I made a regular expression to look through the date string and find the minutes. This worked well, but increased the loop time a tiny bit more. So I then went for the much simpler solution and made a global variable 'minute' from when I make the date string. Added only one line of code. This worked even better.

This configuration has been tested and I have reloaded Verbs with the new code.