Skip to content

Pixel shader fix #1604

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 3 commits into from
Jul 21, 2021
Merged

Pixel shader fix #1604

merged 3 commits into from
Jul 21, 2021

Conversation

FoamyGuy
Copy link
Collaborator

These changes update all projects that use OnDiskBitmap to use the new pixel_shader method from adafruit/circuitpython#4823

These changes are necessary in order to run the projects on CircuitPython 7.0.0.beta-3 and newer.

Once support for CircuitPython 6.x is dropped these could be amended further to remove the getattr call and instead just directly pass bitmap.pixel_shader In the meantime the changes in this PR allow the projects to work on both older versions and newer versions of CircuitPython

Copy link
Contributor

@jepler jepler left a comment

Choose a reason for hiding this comment

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

I skimmed and didn't see any problems with the changes. However, there are merge conflicts which will need to be addressed, I think because I also merged some PRs to fix the max group size. This may indicate needing to work more closely with @lesamouraipourpre when both kinds of updates affect a single file. In any case, thank you!

@FoamyGuy
Copy link
Collaborator Author

I will resolve the merge conflicts tonight to get this back to passing.

# Conflicts:
#	Buckaroo_Plant_Care_Bot/buckaroo_plant_care_bot.py
Copy link
Contributor

@jepler jepler left a comment

Choose a reason for hiding this comment

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

Thank you!

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