Skip to content

Add new Vehicle.CurrentLocation.Coordinates#13

Merged
elffjs merged 1 commit intomainfrom
pr-3852-vss-package-field-for-location
Aug 22, 2025
Merged

Add new Vehicle.CurrentLocation.Coordinates#13
elffjs merged 1 commit intomainfrom
pr-3852-vss-package-field-for-location

Conversation

@elffjs
Copy link
Copy Markdown
Member

@elffjs elffjs commented Aug 22, 2025

This has a new custom struct type, denoted by Types.DIMO.Coordinates. When I run

make venv
source .venv/vss-tools/bin/activate
make csv

I get some new rows:

"Node","Type","DataType","Deprecated","Unit","Min","Max","Desc","Comment","Allowed","Default","Id"
"Vehicle.CurrentLocation.Coordinates","sensor","Types.DIMO.Coordinates","","","","","Current location of the vehicle in WGS 84 coordinates.","","","","7fe502d879175cd5a85353d036e47a66"
"Types.DIMO.Coordinates","struct","","","","","","A struct datatype representing GNSS position.","","","","eda1e3476a3f5bda868c618c26654a2b"
"Types.DIMO.Coordinates.Latitude","property","double","","degrees","-90","90","Latitude in WGS 84 coordinates.","","","","92623867ef575aad848a8ad87f30ac2a"
"Types.DIMO.Coordinates.Longitude","property","double","","degrees","-180","180","Longitude in WGS 84 coordinates.","","","","37bd9a0850205a01b3f31a09e4bb7a1c"
"Types.DIMO.Coordinates.HDOP","property","double","","ratio","0","","Horizontal dilution of position. May be zero if this was not measured.","","","","53fd9d707b1153e796f4a65e21c9808b"

That first non-header row is the one we really want. I don't plan to create a general VSS struct → Go struct regime.

This has a new custom struct type.
@elffjs elffjs requested a review from KevinJoiner as a code owner August 22, 2025 18:41
@linear
Copy link
Copy Markdown

linear bot commented Aug 22, 2025

@elffjs elffjs changed the title Add new currentLocationCoordinates field Add new Vehicle.CurrentLocation.Coordinates Aug 22, 2025
@elffjs elffjs merged commit d348a59 into main Aug 22, 2025
1 of 5 checks passed
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

Successfully merging this pull request may close these issues.

2 participants