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

Feature/test1 #22

Open
wants to merge 42 commits into
base: master
Choose a base branch
from

Conversation

paul-adam
Copy link

Hallo,
I would like you to add the codal-core framework for gyroscope and temperature measurement.
Best regards
Paul

@jamesadevine
Copy link
Contributor

Hi @paul-adam,

Looks great, however one observation is that quite a lot of these API's look the same. For instance, the temperature, humidity and pressure sensors all have the same API's. In addition, commonly this type of sensor is analog, and hence requires a Pin to operate --- for these types of sensor we use the AnalogSensor class, see AnalogSensor.h.

So I have a couple of questions:

  1. Are any of the APIs different from the AnalogSensor api?
  2. Do they require an I2C interface, or are they analog sensors residing on a Pin?

I'd also like to understand a bit more about what you'd like to achieve, just so I can assist you better 😄

@jamesadevine
Copy link
Contributor

@paul-adam the sensor work looks awesome! Really pleased with how that dropped out to just have one function basically other than the constructor 😄

How does this generalise to the I2C? Have you given it a go?

@pelikhan
Copy link
Contributor

Support for Adafruit Seesaw?

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