You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Given the #126 issue was most likely the result of accidentally using milliseconds instead of seconds, there should be something that can validate whether or not values in the file make sense.
On a related note, the use of different units of measurement (seconds vs milliseconds) within the same file is probably a bad move since that can definitely be confusing. The reason that was done was explicitly because, at the time, it felt confusing to have to convert to milliseconds for values that were always and would conceivably always be in seconds. For example, an ITI would be pretty odd if it was 1.530 seconds. On the other hand, a USDelay could conceivably be within millisecond ranges less than 1 second or some other interval.
I don't know what is best. But should be considered for any future projects/configurations of things...
The text was updated successfully, but these errors were encountered:
Given the #126 issue was most likely the result of accidentally using milliseconds instead of seconds, there should be something that can validate whether or not values in the file make sense.
On a related note, the use of different units of measurement (seconds vs milliseconds) within the same file is probably a bad move since that can definitely be confusing. The reason that was done was explicitly because, at the time, it felt confusing to have to convert to milliseconds for values that were always and would conceivably always be in seconds. For example, an ITI would be pretty odd if it was 1.530 seconds. On the other hand, a USDelay could conceivably be within millisecond ranges less than 1 second or some other interval.
I don't know what is best. But should be considered for any future projects/configurations of things...
The text was updated successfully, but these errors were encountered: