Since mym doesn't support NULLs for bigint fields anymore (for good reasons, see https://github.com/datajoint/mym/pull/1), we need to get rid of them in the database as well.
I suggest we simply use zero instead of NULL. Unless I'm overlooking something, all we need to do is update cleanup() to test for __stop_time == 0 instead of isnan(__stop_time) and update the LabView programs to insert zero instead of leaving it NULL.
Both changes need to be done simultaneously and should be tested. I also created an issue in labview-acquisition.
aecker
commented
10 years ago
Since mym doesn't support NULLs for bigint fields anymore (for good reasons, see https://github.com/datajoint/mym/pull/1), we need to get rid of them in the database as well.
I suggest we simply use zero instead of NULL. Unless I'm overlooking something, all we need to do is update cleanup() to test for __stop_time == 0 instead of isnan(__stop_time) and update the LabView programs to insert zero instead of leaving it NULL.
Both changes need to be done simultaneously and should be tested. I also created an issue in labview-acquisition.