New/unknown error message

[alexpiet]

Describe the bug
When loading the trials data frame for a session, I get a new warning and error message.

To Reproduce

Expected behavior
Load without errors or warning

Actual Behavior

Environment (please complete the following information):

• OS & version: Centos 7
• Python version 3
• AllenSDK version 1.6.0

@kschelonka @wbwakeman

[alexpiet]

Email from Doug:

Alex, Kat has been working recently to improve the algorithm for display lag calculation. She has a PR on the SDK that I believe is close to ready to merge in. It’s here:
#1438
I’m not sure how much thought you’ve given to the display lag calculation, so forgive me if I’m stating the obvious, but there’s a square in the corner of the screen that is set to toggle between black/white every 60 visual frames. A photodiode is monitoring that square to detect those transitions. The old algorithm was only looking at a single direction of the edges (black-to-white or vice versa, but not both), so the number of measured transitions would often (I’m assuming half the time) end up being off-by-one. It looks like that’s what you encountered. I’m not quite sure why you’d just now be seeing this warning, though. I’m also not quite sure how the default value was changed, though this value looks more inline with values that Kat has been calculating with her improved algorithm.

[kschelonka]

Hi Alex,

To add to what Doug said -- before this coming update, we changed the default monitor delay calculation to be more in line with reality. You're seeing the warning text because we updated the code so that error messasges are no longer suppressed (previously they were). In the coming update, we are removing the default calculation, but cases like this example will be calculated properly.

[alexpiet]

Thanks @kschelonka !