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

Implement Integer readout on newer SPS firmware to avoid I2C buffer issues #24

Open
winkj opened this issue Oct 31, 2020 · 1 comment
Open
Labels
enhancement New feature or request

Comments

@winkj
Copy link
Member

winkj commented Oct 31, 2020

No description provided.

@winkj winkj added the enhancement New feature or request label Oct 31, 2020
@matthijskooijman
Copy link

On additional problem is the serial number, which can be up to 32 bytes (so 48 bytes including checksums) long (though I think it is shorter in practice, so with some smartness in the code, even a short read can probably still return success if it reads at least one NUL byte and then assume all missed bytes would also have been NUL).

As for reading integers instead of floats, IIUC, this just rounds the values to integers, right? Would have been nice if a fixed-point format was used, since only 12 bits are needed for the supported 0-3000 #/cm³, so that could have left 4 bits for a fixed point part. OTOH, the actual precision is ±10 at best, so rounding to integers probably won't lose any really meaningful info anyway.

matthijskooijman added a commit to matthijskooijman/ArduinoCore-stm32l0 that referenced this issue Nov 2, 2020
The old value of 32 bytes is too small to use the Sensirion SPS030
sensor with the current version of the library for that. See:
Sensirion/arduino-sps#24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants