Skip to content

Fix #52 Distinguish stride from width for SSD1680 #53

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

Merged
merged 1 commit into from
Jun 22, 2021
Merged

Fix #52 Distinguish stride from width for SSD1680 #53

merged 1 commit into from
Jun 22, 2021

Conversation

cdmuhlb
Copy link
Contributor

@cdmuhlb cdmuhlb commented Jun 12, 2021

Apply fix from 8022b34 to Adafruit_SSD1680, allowing rotations of 1 and
2 to be used even when width is not a multiple of 8.

@cdmuhlb
Copy link
Contributor Author

cdmuhlb commented Jun 12, 2021

Testing

Fix verified for rotations 1 & 2 on a Raspberry Pi Zero W with Adafruit 2.13" E-ink Bonnet (graphics no longer shifted by 4 pixels). No regressions encountered with rotations 0 & 3.

@ladyada
Copy link
Member

ladyada commented Jun 12, 2021

nice! thank you for the PR :)

@ladyada ladyada requested a review from makermelissa June 12, 2021 16:05
Apply fix from 8022b34 to Adafruit_SSD1680, allowing rotations of 1 and
2 to be used even when width is not a multiple of 8.
@cdmuhlb
Copy link
Contributor Author

cdmuhlb commented Jun 12, 2021

It looks like I neglected to cede control over hand formatting. Revised fix, reformatted by Black, in work.

Copy link
Member

@tannewt tannewt left a 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!

@tannewt tannewt merged commit 042a433 into adafruit:main Jun 22, 2021
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Jun 25, 2021
Updating https://github.com/adafruit/Adafruit_CircuitPython_EPD to 2.9.4 from 2.9.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_EPD#53 from cdmuhlb/issues/52-ssd1680-stride
  > Moved default branch to main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_IS31FL3731 to 3.2.0 from 3.1.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_IS31FL3731#45 from lesamouraipourpre/matrix5x5
  > Moved default branch to main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_LC709203F to 2.1.0 from 2.0.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_LC709203F#14 from jposada202020/correcting_docs
  > Merge pull request adafruit/Adafruit_CircuitPython_LC709203F#13 from dgriswo/main
  > Moved default branch to main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template
  > "Increase duplicate code check threshold "

Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Text to 2.18.5 from 2.18.4:
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Text#152 from jposada202020/correcting_bitmaplabel_anchorpoint
  > Moved default branch to main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA:
  > Added the following libraries: Adafruit_CircuitPython_NeoKey
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.

3 participants