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

expand on remote_devices.yaml #704

Open
bimac opened this issue Aug 7, 2024 · 0 comments
Open

expand on remote_devices.yaml #704

bimac opened this issue Aug 7, 2024 · 0 comments
Assignees

Comments

@bimac
Copy link
Contributor

bimac commented Aug 7, 2024

remote_devices.yaml currently represents a dict with device names as keys and the udp address as values:

camera: udp://[IP]:[PORT]

There are a few issues with this approach:

  • The current layout does not allow adding further properties in the future. It might make sense to nest the dict:

    unique identifier:
       name: camera
       address: udp://[IP]:[PORT]
       property1: xyz
       property2: xyz
    
  • Using the name as an identifiers is prone to collisions (if, say, there are two computers identifying as camera). It might make sense to go with the device's unique MAC address?

    58:11:22:b1:62:a5:
       name: camera
       address: udp://[IP]:[PORT]
    

    The MAC address can be obtained like so:

    key = ':'.join(re.findall('..', '%012x' % uuid.getnode()))
    

    This way, machines could update their entries even in the face of dynamic IP addresses.

  • A date-field indicating the respective devices last announcement could be useful for cleaning up the list of devices

@bimac bimac assigned bimac and k1o0 Aug 7, 2024
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

2 participants