Closed malaakso closed 7 years ago
Sorry for jumping to conclusions, I see all previous PRs raised against "dev". So, changed the base branch back as before Although, my doubt still remains, any use of maintaining 2 branches?
@malaakso Sorry about the delay, I haven't had access to any of the clicks until today, I'll do a review and verify the PR as soon as I can!
@nikhil-zinjurde-imgtec Dev is for whatever is done between releases that shouldn't be pulled in, sometimes contains design changes that I would prefer people not checking out until a release have been made. Master is always safe to check out, even if it's ahead of the newest release since it sometimes contains some minor fixes (for CI etc.).
Code looks good, unfortunately the only weather click I have access to is not working properly due to I2C issues so I can't properly test it. According to the docs this looks fine though, if any issues pop up due to this change it will need a fix in LMC.
PIC32MX doesn't have FPU, so it makes sense to perform BME280 compensation calculations using integer math. Accuracy is unaffected.