Migrate several widgets to the data-lake#2449
Open
rafaellehmkuhl wants to merge 9 commits intobluerobotics:masterfrom
Open
Migrate several widgets to the data-lake#2449rafaellehmkuhl wants to merge 9 commits intobluerobotics:masterfrom
rafaellehmkuhl wants to merge 9 commits intobluerobotics:masterfrom
Conversation
03b04af to
00f9871
Compare
Replaces the vehicle store attitude watch with useDataLakeVariable composable. Roll and pitch are now read from configurable data lake variable IDs (default: /mavlink/1/1/ATTITUDE/roll and pitch). Removes useMainVehicleStore dependency.
Replaces the vehicle store attitude watch with useDataLakeVariable composable for yaw. The variable ID is now configurable via widget.options.yawVariableId (default: /mavlink/1/1/ATTITUDE/yaw). Removes useMainVehicleStore dependency.
Replaces vehicle store watches for roll, pitch, and camera tilt with useDataLakeVariable composable. All three variable IDs are now configurable via widget options. Also fixes the broken genericVariables watch by reading cameraTiltDeg directly from the data lake. Removes useMainVehicleStore dependency.
Replaces the vehicle store altitude watch with useDataLakeVariable composable. The depth source variable ID is now configurable via miniWidget.options.depthVariableId (default: /mavlink/1/1/AHRS2/altitude). Properly accepts MiniWidget props. Removes useMainVehicleStore dependency.
Replaces the vehicle store altitude watch with useDataLakeVariable composable. The variable ID is configurable via miniWidget.options. altitudeVariableId (default: /mavlink/1/1/GLOBAL_POSITION_INT/relative_alt). Raw value is in mm from MAVLink, converted to meters by dividing by 1000. Properly accepts MiniWidget props. Removes useMainVehicleStore dependency.
2ad8810 to
a541990
Compare
Replaces vehicle store powerSupply/instantaneousWatts access with useDataLakeVariable composable. Voltage, current, and remaining are now read from configurable data lake variable IDs. Raw MAVLink units (mV, cA, %) are converted in local computed properties. Watts is computed locally as voltage * current instead of reading from the store. Removes useMainVehicleStore dependency. Default variable IDs: - voltageVariableId: /mavlink/1/1/SYS_STATUS/voltage_battery - currentVariableId: /mavlink/1/1/SYS_STATUS/current_battery - remainingVariableId: /mavlink/1/1/SYS_STATUS/battery_remaining
a541990 to
f774b22
Compare
This file contains hidden or 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
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.
Depends on #2441 and #2448.
Helps with milestone 17 (relevant to #2153, #2154, and #2155).
I'm initially moving them to consume from the data-lake, and I will them modify them to allow for the selection of the desired source (variable).