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

BMP180 pressure and temp sensor support. #32

Closed
wants to merge 7 commits into from

Conversation

MrSpock
Copy link

@MrSpock MrSpock commented Mar 15, 2017

Hi,
I needed support for BMP180 and I needed to lear Rust. Here is result :)
Tested on my real BMP180 against other library (same results) and test scenario based on BMP180 documentation.

@Kixunil
Copy link

Kixunil commented Mar 15, 2017

I'm afraid you missed #27. But shouldn't be much of a problem.

@MrSpock
Copy link
Author

MrSpock commented Mar 15, 2017

Yes, definitely I missed that one :) Single file so yes - no problem to extract that to separate crate.

@posborne
Copy link
Member

Awesome, thanks for the contribution. Been busy lately, but hoping to get back to being a better maintainer here or adding other maintainers that will be able to drive things more actively.

@tadeegan
Copy link

Did you ever extract this to its own crate? I'm interested in using it :D.

@Kixunil
Copy link

Kixunil commented Jun 10, 2017

As I wrote in #27, I've started working on it.

What needs to be done:

  • Decide where to put the sensor traits (Thermometer, Accelerometer...)
  • Create and publish the crate for the traits
  • Who should be the owner (of crates, repo)?
  • Licenses
  • Fill info into Cargo.toml for each sensor and publish them.
  • Merge Removed sensors, so they can be put in separate crate #28

@posborne
Copy link
Member

Closing this as we are moving away from adding additional sensors/traits to this crate in favor of building those on top of embedded-hal traits.

@posborne posborne closed this Mar 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants