Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

As a user, I expect Pressure in NMEA sentence to be compatible with OpenCPN #69

Closed
sarfata opened this issue Nov 23, 2017 · 1 comment · Fixed by #121
Closed

As a user, I expect Pressure in NMEA sentence to be compatible with OpenCPN #69

sarfata opened this issue Nov 23, 2017 · 1 comment · Fixed by #121

Comments

@sarfata
Copy link
Owner

sarfata commented Nov 23, 2017

OpenCPN doc says:

Measured Value Transducer Type Measured Data Unit of measure Transducer Name
barometric "P" pressure 0.8..1.1 or 800..1100 "B" bar "Barometer"

We should be publishing in Bars, which should work but Ron had to make a change to get this to work:
https://github.com/sarfata/kbox-firmware/pull/61/files#diff-e1a852b1882fac49cb1e32e9192d85e1R57

@ronzeiller
Copy link
Contributor

I think Ron is right.
In KBox the sensor gives Pascal and also Signal K is Pascal which is ok.
The NMEA0183 building gets Pascal too, but here it should be bar
So 1/100000 should be right

May be also a new define in SKUnits.h?
#define SKPascalToBar(x) x / 1000000

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants