Added Dashboard clear button #121
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
KavinSatheeskumar
approved these changes
Aug 25, 2023
There was a problem hiding this comment.
Reviewable status: 0 of 1 files reviewed, all discussions resolved
Darth-Raazi
added a commit
that referenced
this pull request
Oct 20, 2023
* proof of concept works * Misc WDR fixes * dynamic text dash item, fix errors and dynamic sizing * More fixes * Limit title length * Found better alternative than QTimer now we listen to all can messages (acts as our interrupt function) and then check if the specified peiod has changed, in which case, send the can message. * Update DAQ config * Live fixes * Add printing to replay log * Fix parsley performance * Remove CAN message table * Clean up wdr nitpicks * Clean up parsley source * Fix unit tests, autopep8 * Add duplicate item button, optional filter on dynamic text and sorted item keys (#111) * Update RLCS source * RLCS source improvements * Update parsley for live telemetry * Update requirements.txt to fix imageitem * Let RLCS control valves * Add better dashboard error and reset handling * Autocomplete dashitems * add general board table item * wip: use QItemDelegate to add dropdown instead * improve copy pasting and add deleting to general boards widget * finish new dropdown implementation and add save/restore support * improve finding list of paths * make table rows draggable * rename general boards to table view * prevent cell deletion when none is selected * add store/restore header for table view and ctrl-x for cut * add support for binary telemetry message * wip: switch to new binary encoding for telemetry * support for new binary encoding * Add dynamic_text background colour change (#113) * dynamic_text: Added button param to make new params * dynamic_text: Add conditions to change background color * dynamic_text: Made requested changes * dynamic_text: Removed float casting to allow CAN messages * dynamic_text: Casting condition param to str for comparison * correct typo * Format --------- Co-authored-by: Kavin Satheeskumar <[email protected]> Co-authored-by: Kavin <[email protected]> * add support for binary telemetry message * wip: switch to new binary encoding for telemetry * support for new binary encoding * fix some bugs * Add the ability to select which parsley instance you want to send to * omnibus to gpsd converter * Revert "Add the ability to select which parsley instance you want to send to" This reverts commit 923bcb4. * Drive changes Fixes a few bugs, light mode, adds parsley selection * Fix parsley * change parsely names to telemetry/none-telemetry; add keep alive bytes for telemetry * increase table widget margin * remove commented code project-wide (#117) * Fix exception caused by empty series * Fixed plot scaling issue (#119) Fixed plot scaling issue, removed broken tests and downgraded parsley. * Added and fixed clear button (#121) Added a dashboard 'Clear' button * Deal with 1920x1080 screen resolution on Wayland (#143) * Another Wayland fix (for dynamic text) (#144) * Deal with 1920x1080 screen resolution on Wayland * Fix dynamic text for Wayland * Removed flakey replay_log unit test * Ran format.sh to enforce autopep --------- Co-authored-by: Michael Li <[email protected]> Co-authored-by: DAQ Computer <[email protected]> Co-authored-by: Michael <[email protected]> Co-authored-by: Kavin Satheeskumar <[email protected]> Co-authored-by: Robert Cai <[email protected]> Co-authored-by: Rio Liu <[email protected]> Co-authored-by: Rio <[email protected]> Co-authored-by: Fayzulloh Ergashev <[email protected]> Co-authored-by: Kavin <[email protected]> Co-authored-by: Anthony Chen <[email protected]> Co-authored-by: MaisimZaman <[email protected]> Co-authored-by: Ozayr Raazi <[email protected]>
Darth-Raazi
added a commit
that referenced
this pull request
Oct 24, 2023
* Rewrote launcher to support macOS * Fixed KeyError * Added prompt menu and new profiles * Terminate all processes when any of them exits * Added stderr printing * Updated stdout and stderr printing, changed how programs are terminated * Added signal for Windows * Potential fix for signal on Windows * I hate Windows * proof of concept works * Misc WDR fixes * dynamic text dash item, fix errors and dynamic sizing * More fixes * Limit title length * Found better alternative than QTimer now we listen to all can messages (acts as our interrupt function) and then check if the specified peiod has changed, in which case, send the can message. * Update DAQ config * Live fixes * Add printing to replay log * Fix parsley performance * Remove CAN message table * Clean up wdr nitpicks * Clean up parsley source * Fix unit tests, autopep8 * Add duplicate item button, optional filter on dynamic text and sorted item keys (#111) * Update RLCS source * RLCS source improvements * Update parsley for live telemetry * Update requirements.txt to fix imageitem * Let RLCS control valves * Add better dashboard error and reset handling * Autocomplete dashitems * add general board table item * wip: use QItemDelegate to add dropdown instead * improve copy pasting and add deleting to general boards widget * finish new dropdown implementation and add save/restore support * improve finding list of paths * make table rows draggable * rename general boards to table view * prevent cell deletion when none is selected * add store/restore header for table view and ctrl-x for cut * add support for binary telemetry message * wip: switch to new binary encoding for telemetry * support for new binary encoding * Add dynamic_text background colour change (#113) * dynamic_text: Added button param to make new params * dynamic_text: Add conditions to change background color * dynamic_text: Made requested changes * dynamic_text: Removed float casting to allow CAN messages * dynamic_text: Casting condition param to str for comparison * correct typo * Format --------- Co-authored-by: Kavin Satheeskumar <[email protected]> Co-authored-by: Kavin <[email protected]> * add support for binary telemetry message * wip: switch to new binary encoding for telemetry * support for new binary encoding * fix some bugs * Add the ability to select which parsley instance you want to send to * omnibus to gpsd converter * Revert "Add the ability to select which parsley instance you want to send to" This reverts commit 923bcb4. * Drive changes Fixes a few bugs, light mode, adds parsley selection * Fix parsley * change parsely names to telemetry/none-telemetry; add keep alive bytes for telemetry * increase table widget margin * remove commented code project-wide (#117) * Fix exception caused by empty series * Fixed plot scaling issue (#119) Fixed plot scaling issue, removed broken tests and downgraded parsley. * Added and fixed clear button (#121) Added a dashboard 'Clear' button * Rewrote launcher to support macOS * Fixed KeyError * Added prompt menu and new profiles * Terminate all processes when any of them exits * Added stderr printing * Updated stdout and stderr printing, changed how programs are terminated * Added signal for Windows * Potential fix for signal on Windows * I hate Windows * Rewrote launcher to dynamically get all sources and sinks * Added comment * Deal with 1920x1080 screen resolution on Wayland (#143) * Another Wayland fix (for dynamic text) (#144) * Deal with 1920x1080 screen resolution on Wayland * Fix dynamic text for Wayland * Added finally section to ensure all spawned process are killed no matter how the program exits * Removed flakey replay_log unit test * Ran format.sh to enforce autopep * Python executable and signal changes for Windows --------- Co-authored-by: Michael Li <[email protected]> Co-authored-by: BluCodeGH <[email protected]> Co-authored-by: DAQ Computer <[email protected]> Co-authored-by: Michael <[email protected]> Co-authored-by: Kavin Satheeskumar <[email protected]> Co-authored-by: Robert Cai <[email protected]> Co-authored-by: Rio Liu <[email protected]> Co-authored-by: Rio <[email protected]> Co-authored-by: Fayzulloh Ergashev <[email protected]> Co-authored-by: Kavin <[email protected]> Co-authored-by: Anthony Chen <[email protected]> Co-authored-by: MaisimZaman <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update from my last pull request to add a functional dashboard clear button. Named 'Clear'
This change is