5g-era / reference-netapp Goto Github PK

The client examples should unregister from the server once the input video has ended.

Adapt the /image endpoint based on this issue 5G-ERA/era-5g-client#28

• Update examples README.md
• Update era_5g_network_application_template
• Update distribution version stuff.
• Update and test or remove docker files.
• Remove/update ROS 1 stuff?
• Test ROS 2 stuff
• Remove/update k8_config

There should be an API available to check the processing state of the client's task - at least to find out if there are any unprocessed data from the client.

Adapt the reference network application to reflect the changes in the client library regarding the producer/consumer model.

• Update register method (add the id from the client)
• Do not distribute the results to the clients who serve only as a producer
• Distribute results to all consumers with the same ID

If the client connects to an incompatible netapp, e.g. one already running on the same port, it has no feedback from it and continues to send image data even if it does not receive results.

• Pokud se ukončí klient natvrdo (SIGKILL), tak zůstane v interface obsazený port, a také možná zůstanou viset nějaký vlákna
• Až skončí video, bylo by asi dobré odregistrovat se.
• Celkově asi nejsou otestovány a vyřešeny situace, kdy něco spadne, přeruší se spojení nebo něco zůstane viset.
• Pokud jede klient i NetApp, pak se NetApp a znovu zapne, tak se klient asi neodpojí a snaží se stále připojit.
• Port pro socketio by měl být parametrizovatelný (při spouštění NetApp), a také by měl jít získat z middleware (pokud se používá), stejně jako je teď získána adresa. Teď je u klienta v kódu natvrdo.
• U příkladů klienta ("test_client_*") chcete např. cestu k testovacímu videu nastavit přes enviromentální proměnné. S tím ve finální verzi nemám problém, ale nechal bych tam nějaké záložní defaultní nastavení, aby to prostě nějak fungovalo hned po stažení z gitu a instalaci balíčků. Přijde mi otravné pro první spuštění zkoumat, jaké musím nastavit proměnné. Pokud by to mělo být uloženo v env proměnných, tak bych tam dal i podobné věci, jako IP, login, heslo, task_id, ale také cesty ke configu sítí yolo, mask_rcnn apod. (ty jsou tam teď také natvrdo). Možná by bylo dobré udělat místo env proměnných možná nějaký konfigurační soubor. Pěkné by to bylo na jednom místě.
• Vypadá to, že do requirements se budou ještě muset přidat balíčky pro mmdet, mmvc apod., ale k prozkoumání těchto zdrojáků jsem se ještě pořádně nedostal.
• Pokud se aktuální kódy mají dále využívat pro jiné NetApp a balit do balíčků, tak za mě by to chtělo určitě ještě alespoň jednou tolik okomentovat nebo lépe pojmenovat některé proměnné a prostě trochu více učesat. Těmi kódy procházím už několik dnů, mám toho plnou hlavu a rozklíčovat fungování některých částí je celkem peklo.

May be the test video and the test image from assets dir should be moved to the separate client repository (era-5g-client), so that it is together with the client code?

When the client examples are killed using SIGKILL, an unused port remains taken on the server and (perhaps) some threads are not correctly destroyed.