-
Notifications
You must be signed in to change notification settings - Fork 13
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
Replace Measurement
and UnscaledMeasurement
with explicit Acceleration
and MagneticField
structs.
#12
Conversation
474b7f6
to
e451306
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice improvements, thank you!
Overall it looks very good to me. I would just skip bitflags
as described in the other issue.
Could you also:
- Add tests for the temperature enable/disable methods
- Add tests for the
raw
version of the measurement methods - Add an entry to the changelog?
Added tests for the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Could you also update the method list in the README as well?
Other than that this would be good to go.
Ugh, GH is showing a bunch of conflicts now that I merged #11 but git should be able to trivially merge them as I explicitly did not squash the commits.
…nd `MagneticField`.
144c1c9
to
d692da6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thank you for your work!
Simplify the API by having only one method each for measuring, with the conversion (unscaled/mg/nT) being done by the explicit types.
Also included the changes from #11.