Skip to content

Conversation

@Yury-MonZon
Copy link
Contributor

@Yury-MonZon Yury-MonZon commented Jan 19, 2024

Some of us prefer to see wind speed in m/s, not in km/h. This PR does it:
-m/s icon on OSD with corresponding speed in m/s
-CLI parameter osd_estimations_wind_mps , OFF by default.
-explanation in docs/Settings.md

Enabled only in OSD_UNIT_METRIC mode. Tested in flight (vision font).
image

@DzikuVx DzikuVx added this to the 8.0 milestone Jan 22, 2024
Copy link
Member

@DzikuVx DzikuVx left a comment

Choose a reason for hiding this comment

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

@Yury-MonZon thanks for new feature. There is one thing that has to be improved thought.
After changing configuration structure, parameter group version has to be changed as well or flash will be broken after upgrade. Please see https://github.com/iNavFlight/inav/pull/9652/files#diff-a680ca1e23901579e9214c055509004e8b5c05e3f7281ed74ac1edfc38caabaeL212

@Yury-MonZon
Copy link
Contributor Author

Thanks Pawel.

@DzikuVx DzikuVx merged commit d1a8469 into iNavFlight:master Apr 4, 2024
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