Skip to content

Added elevation data on simulator side#22

Merged
imisaacwu merged 4 commits intomainfrom
Elevation-data
Apr 11, 2025
Merged

Added elevation data on simulator side#22
imisaacwu merged 4 commits intomainfrom
Elevation-data

Conversation

@PatrickRung
Copy link
Copy Markdown
Collaborator

Changed GPS sensor data to contain information about elevation data and for that data to be sent over in the positionReport Json.

Changed GPS sensor data to contain information about elevation data. However the elevation might not be in right units.
Copy link
Copy Markdown
Collaborator

@imisaacwu imisaacwu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good so far 👍🏻

Comment thread Assets/Scripts/GpsSensor.cs Outdated
Comment thread Assets/Scripts/GpsSensor.cs
Comment thread Assets/Scripts/GpsSensor.cs
Copy link
Copy Markdown
Collaborator

@imisaacwu imisaacwu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

finicky formatting :P

Comment thread Assets/Scripts/GpsSensor.cs Outdated
Comment thread Assets/Scripts/GpsSensor.cs Outdated
@imisaacwu imisaacwu self-requested a review November 22, 2024 21:16
Copy link
Copy Markdown
Collaborator

@imisaacwu imisaacwu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 👍

@imisaacwu imisaacwu merged commit c510041 into main Apr 11, 2025
@imisaacwu imisaacwu deleted the Elevation-data branch April 11, 2025 21:59
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