Skip to content

Conversation

@jepler
Copy link
Contributor

@jepler jepler commented May 12, 2021

and also check the product ID as suggested by @jerryneedell from the Arduino example.

The change to the initial value of last_position causes the initial position (usually 0) to be printed each time the program starts.

.. this wraps the core code in seesaw.py into a compatible subset of
the core's rotaryio.IncrementalEncoder class.
@jepler jepler requested a review from kattni May 12, 2021 17:44
Copy link
Contributor

@kattni kattni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this! Question below.


seesaw = Seesaw(i2c_bus, addr=0x36)

seesaw_product = (seesaw.get_product() >> 16) & 0xFFFF
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code is failing here. Where is get_product() coming from?

Copy link
Contributor

@kattni kattni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested successfully on FunHouse with a STEMMA rotary encoder!

@kattni kattni merged commit 6467edb into adafruit:master May 12, 2021
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request May 13, 2021
Updating https://github.com/adafruit/Adafruit_CircuitPython_Seesaw to 1.8.1 from 1.7.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_seesaw#64 from jepler/incrementalencoder
  > Merge pull request adafruit/Adafruit_CircuitPython_seesaw#65 from jepler/use-board-i2c
  > Merge pull request adafruit/Adafruit_CircuitPython_seesaw#62 from kattni/rotary-update
  > Merge pull request adafruit/Adafruit_CircuitPython_seesaw#61 from jfurcean/add-basic-rotary
  > Merge pull request adafruit/Adafruit_CircuitPython_seesaw#60 from rianadon/patch-1
  > "Increase duplicate code check threshold "
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.

2 participants