You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Similar need to #26 , but implementation will differ. Mainly because BaseDevices must be instantiated and held to maintain connections to physical devices.
There are two needs here, both to be supported while ALabOS is running:
edit existing device definitions (main priority)
add/remove devices from the lab (lower priority, as this suggests physical updates to the lab which should imply work stoppage anyways).
Proposed solution for edits:
Create a function reload_device(device_name: str) which disconnects device, reloads device definition, reconnects device
Create a button on the dashboard to execute this function. Maybe require device to be paused first?
Proposed solution for add/remove (less clear tbh but here is an idea):
Create a function to reload the device configuration file, look for changes. On new devices, just load them and add to ALabOS mongodb device collection. On removed devices, push a notification to the GUI "this device is slated for removal, please pause". Once pause status is achieved, it will be dropped from the dashboard + collection.
The text was updated successfully, but these errors were encountered:
Updated plan. We will support device reloading as described above. To add/remove devices from the lab, one must stop ALabOS. This will require a system pause button that will allow ongoing tasks to complete, then a clean stop/restart state so the lab can be reset.
Similar need to #26 , but implementation will differ. Mainly because
BaseDevice
s must be instantiated and held to maintain connections to physical devices.There are two needs here, both to be supported while ALabOS is running:
Proposed solution for edits:
reload_device(device_name: str)
which disconnects device, reloads device definition, reconnects deviceProposed solution for add/remove (less clear tbh but here is an idea):
The text was updated successfully, but these errors were encountered: