Closed msjonker closed 1 year ago
@msjonker thanks for reporting this issue. @danleyb2 is going to make the script more robust.
@msjonker I can confirm this PR fixes the issue with multiple cameras https://github.com/parkpow/deep-license-plate-recognition/pull/125
Only 1 line is tailed from the docker logs command, so when there are multiple cameras (2 in my case), there's a good chance that over a shorter timeout interval (including the default of 20s) that a camera will be shown as offline when it is actively logging, because the line that was tailed over that interval never happened to be a certain camera https://github.com/parkpow/deep-license-plate-recognition/blob/master/stream/stream_monitor.py#L78
IMO it would be great if this monitor was part of the Stream image itself and did not rely on having to parse docker logs.
Example of the issue:
Debug output from stream_monitor.py:
Docker logs for stream over that time period: