Scottapotamas / MS5837

Portable C implementation for the I2C MS5837 pressure sensor
MIT License
1 stars 0 forks source link

Issue due to lack of knowledge (probably) #1

Open mathos00 opened 3 months ago

mathos00 commented 3 months ago

Hello, I've been trying this software, which seems well organized, and readable. I'm working on a volunteer project for the Catalina Hyperbaric chamber to make a rate meter, so we need a very accurate sensor... thus the MS5837.... and I'd like to see if I can get this software to work in that project. But I'm not quite sure why I'm getting this output from the sample Arduino script included:

11:55:43.488 -> Pressure (mbar) = 49.55 11:55:43.488 -> Temperature C = 24.08 11:55:43.488 -> 11:55:44.038 -> Pressure (mbar) = 49.55 11:55:44.038 -> Temperature C = 24.08

Using Bluerobotics library I get this output:

12:35:47.409 -> Temperature: 23.88 deg C 12:35:47.449 -> Altitude: 184.26 m above mean sea level 12:35:48.476 -> Pressure: 990.80 mbar

Scottapotamas commented 3 months ago

I suspect I have an idea of the issue - this library was written and tested against a MS5837-02BA (1200mbar max) and some of the higher pressure rated parts such as MS5837-30BA (30bar max) use slightly different conversion constants.

My implementation didn't cover the other variants when I wrote it.

You can confirm which sensor variant you have by using the data gathered during the calibration constant readout during startup.

I'm happy to add the extra conversion logic to handle your sensor variant, if that's the issue.