ms1450 / CommandConnectorCompatibilityCalculator

Compares a customer list of Camera Models with the Verkada HCL to provide compatible cameras.

2 stars 1 forks source link

[FR] GUI #23

Closed ian-young closed 1 month ago

ian-young commented 1 month ago

GUI

What's included

[x] Compatibility list is printed clearly and with the same or improved formatting
[x] Command connector recommendation is shown
- [x] Excess channels are visible
[x] Modern feeling UI with appropriate padding
[x] Dynamic resizing with the appropriate scroll bars
[x] Drag and drop functionality
- [x] File name is visible
[x] Easy navigation (toolbar menu vs buttons)
- [x] Restructure so calculations are ran only once and passed through tabulation on button press
[x] Scalable
[x] Add how much retention is required to dynamically set the variable
- [x] Warning is present if they attempt to go above 90 or below 30 days.

ian-young commented 1 month ago

Added decorators to the code to enhance logging and ensure consistency in log output. Need to move forward with testing the memory management side of things. The class is created and implemented for the most part in 1ab8c7f.

ian-young commented 1 month ago

Commit 54fde436ecb49a7ad717e72ee78e08498fcb96ca adds the Spinbox to dynamically set retention. Its bounds are set to 30 and 90 which relate to what the Command Connector can do. A pop up may be appropriate if the user goes in and sets a value that violates these bounds.

[!NOTE] Text needs to be added to indicate to the user that the Spinbox is for setting the retention in days.

[!IMPORTANT] It is also important that the user understands that retention needs to be an int value since they cannot set 30.5 days worth of retention for example

ms1450 commented 1 month ago

Added Command Connectors, limitation to the retention period (users can only select 30, 60 or 90 days). Also cleaned up UI.