Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Timestamp mechanism is flawed #172

Open
mischastik opened this issue Dec 7, 2018 · 1 comment
Open

Timestamp mechanism is flawed #172

mischastik opened this issue Dec 7, 2018 · 1 comment

Comments

@mischastik
Copy link
Contributor

Currently, the image timestamp is created by the base class "Camera". Some cameras / SDKs could supply a more accurate timestamp. But it should be ensured that all camera implementations supply a timestamp as UTC ticks.

@mischastik
Copy link
Contributor Author

Discussion on this:
Michael [vor 8 Minuten]
Mit der neuen Lösung bin ich allerdings auch nicht sehr glücklich, weil wir tendenziell gute Zeitstempel von der Kamera wegwerfen.

Hannes [vor 7 Minuten]
Ja, aber schlag mal was besseres vor. Man könnte den TS wieder per default setzen, und die Implementierungen dazu verpflichten, Timestamps in UTC zu setzen. Das is aber sehr schwierig durchzusetzen.

Michael [vor 7 Minuten]
Korrekt für die SICK wäre eigentlich: Beim Connect den Versatz zwischen uptime und UTC speichern und damit jedes mal den "korrekten" UTC timestamp berechnen. Das geht mit der aktuellen implementierung gar nicht mehr

Hannes [vor 6 Minuten]
Der aktuelle Mechanismus ist halt zuverlässig iSv: er tut das, was er soll. Ob das, was er tut, gut ist, ist ne andere Frage.

Hannes [vor 5 Minuten]
Wenn man den Kameras erlaubt, Zeitstempel zu setzen, dann muss man das irgendwie testen, ansonsten wird's wieder Wildwuchs geben.

Michael [vor 4 Minuten]
Eigentlich ist das gar nicht so schwer zumindest grob zu testen.

Michael [vor 4 Minuten]
Es gibt doch schon generische Kameratests. Da könnte man einfach abprüfen, ob der Timestamp eines frames grob der aktuellen UTC-Zeit entspricht.

Hannes [vor 3 Minuten]
Möglichkeit 1 (zuverlässig):
Es gibt einen TS pro Kamera, den die Basisklasse verwaltet. Die Implementierungen können ihn setzen, und die Basisklasse prüft jedes Mal zur Laufzeit, ob er plausibel ist (max. Abweichung zu UtcNow).

Hannes [vor 2 Minuten]
Möglichkeit 2 (weniger zuverlässig):
Man erlaubt so wie früher den Kameras, beliebige TS pro Kanal zu setzen. Dann muss man das mit Tests abfangen, die mit angesteckter Kamera durchgeführt werden müssen

Hannes [vor 2 Minuten]
Solche Tests gibt's noch nicht wirklich, wären aber sehr cool.

Hannes [vor 2 Minuten]
So, ich bin ma wieder afk

Michael [vor 2 Minuten]
ok

Michael [vor 1 Minute]
ist jetzt sowieso nicht dringend. Lassen wir's mal, so wie es ist. Es gab halt schon projekte, wo wir einen ordentlichen Zeitstempel verwendet haben.

Hannes [vor < 1 Minute]
Jap. Wenn das ein Anliegen für dich ist, das mal ordentlich zu machen, dann mach doch ein Ticket in Github auf, und kopier diesen Thread rein, dann haben wir das schonmal gesichert

Michael [vor < 1 Minute]
mach ich

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant