Skip to content

Pixelbuf support #74

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

Pixelbuf support #74

merged 5 commits into from
Apr 30, 2020

Conversation

rhooper
Copy link
Contributor

@rhooper rhooper commented Feb 21, 2020

This MR adds support for _pixelbuf.

Notable is that this means .buf is gone. The way _pixelbuf works now means there is no .buf property. In theory we can emulate that with a @Property.

@tannewt
Copy link
Member

tannewt commented Feb 24, 2020

Thanks for adding this! I'd like to wait to merge until 5.x is stable because 4.x crashes.

@dunkmann00
Copy link
Contributor

Is this good to be merged now that CP is at 5.0?

@dunkmann00
Copy link
Contributor

@kattni Any plans for when this can be merged back in?

@tannewt
Copy link
Member

tannewt commented Mar 30, 2020

@dunkmann00 We don't have any immediate plans though we should be able to do it. Maybe we can once @rhooper dusts off the PR? I may have a variant somewhere here too. It'd be good if it fell back to PyPixelBuf based on the CircuitPython major version instead of whether _pixelbuf is available.

@dunkmann00
Copy link
Contributor

@tannewt I'd love to help out with this if I could? I'm not sure how I would go about that though. Should I fork @rhooper's version and just fix the conflicts/fallback code and make sure it plays well with CP 5.0?

@tannewt
Copy link
Member

tannewt commented Apr 13, 2020

@dunkmann00 That would be awesome! You can create your own branch based on his code and do a separate PR.

I also have these changes I made: master...tannewt:update_for_pixelbuf

The main thing is to ensure it works well in 5.x and uses pypixelbuf in 4.x rather than crashing.

Thanks!

@dunkmann00 dunkmann00 mentioned this pull request Apr 24, 2020
@tannewt tannewt merged commit f363100 into adafruit:master Apr 30, 2020
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