Skip to content

Pixelbuf Support #2 #82

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 7 commits into from
Apr 30, 2020
Merged

Pixelbuf Support #2 #82

merged 7 commits into from
Apr 30, 2020

Conversation

dunkmann00
Copy link
Contributor

@dunkmann00 dunkmann00 commented Apr 24, 2020

This adds support for CP _pixelbuf. This aims to complete PR #74.

Works in CP versions 5 and 4 (fallback to pure python Pixelbuf impl.). In order to support the pure python version, PR #18 from the Adafruit_CircuitPython_Pypixelbuf repo is also necessary for this to work.

@dunkmann00
Copy link
Contributor Author

The failed test is addressed in Adafruit_CircuitPython_Pypixelbuf #18

@tannewt tannewt self-requested a review April 24, 2020 17:34
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.

This looks good. Will wait to approve until the Pixelbuf support is merged. Thanks again!

@tannewt
Copy link
Member

tannewt commented Apr 27, 2020

PixelBuf updates are merged but cannot be released until adafruit/Adafruit_CircuitPython_NeoPixel_SPI#14 is resolved and therefore block this PR from being released.

@tannewt
Copy link
Member

tannewt commented Apr 30, 2020

Ok, PyPixelBuf 2.0.0 and NeoPixel_SPI 0.8.0 have been released. I'm going to merge this now and merge once I confirm the other two are on pypi.

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.

Thanks for polishing this up @dunkmann00! Thanks to @rhooper for PixelBuf. 🎉

@tannewt tannewt merged commit 326fed4 into adafruit:master Apr 30, 2020
@ladyada
Copy link
Member

ladyada commented Apr 30, 2020

awesome!!

adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request May 1, 2020
Updating https://github.com/adafruit/Adafruit_CircuitPython_NeoPixel to 6.0.0 from 5.1.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_NeoPixel#83 from adafruit/tannewt-patch-1
  > Merge pull request adafruit/Adafruit_CircuitPython_NeoPixel#82 from dunkmann00/pixelbuf

Updating https://github.com/adafruit/Adafruit_CircuitPython_NeoPixel_SPI to 0.8.0 from 0.4.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_NeoPixel_SPI#15 from dunkmann00/pypixelbuf-2.0-compat

Updating https://github.com/adafruit/Adafruit_CircuitPython_AzureIoT to 2.0.0 from 1.1.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_AzureIoT#11 from jimbobbennett/master
  > build.yml: add black formatting check

Updating https://github.com/adafruit/Adafruit_CircuitPython_Pypixelbuf to 2.0.0 from 1.0.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_Pypixelbuf#19 from dunkmann00/dotstar_brightness
  > Merge pull request adafruit/Adafruit_CircuitPython_Pypixelbuf#18 from dunkmann00/_pixelbuf_cp
  > Merge pull request adafruit/Adafruit_CircuitPython_Pypixelbuf#17 from adafruit/black-update
  > build.yml: add black formatting check
  > Merge pull request adafruit/Adafruit_CircuitPython_Pypixelbuf#16 from kattni/update-example-name
  > Merge pull request adafruit/Adafruit_CircuitPython_Pypixelbuf#12 from adafruit/pylint-update
  > update code of coduct: discord moderation contact section
  > Merge pull request adafruit/Adafruit_CircuitPython_Pypixelbuf#11 from sommersoft/patch_coc
  > update pylintrc for black
@dunkmann00 dunkmann00 deleted the pixelbuf branch May 2, 2020 01:42
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