search

adafruit / Adafruit_CircuitPython_PyBadger

Badge-focused CircuitPython helper library for PyBadge, PyBadge LC, PyGamer and CLUE
MIT License
21 stars 26 forks source link

Update the pixel_shader usage of OnDiskBitmap #46

Closed lesamouraipourpre closed 3 years ago

lesamouraipourpre commented 3 years ago

OnDiskBitmap has had incompatible changes in 7.0.0-alpha.3 - Ref: https://github.com/adafruit/circuitpython/pull/4823 This PR is part of a group of updates for this change - https://github.com/adafruit/circuitpython/issues/4982

This PR updates the library to the combined usage for CP6 & CP7.

Also: Refactor code referencing CircuitPython 4 & 5 Filled out requirements.txt Include most of the packages in api.rst. I can't get docs to build for cpb_gizmo so have left it out of the docs for now.

It has not been tested as I do not have the hardware. I tried testing with the PyPortal example but I am getting this bug (even before changes): https://github.com/adafruit/circuitpython/issues/4985

lesamouraipourpre commented 3 years ago

I've tried pulling this PR into a clean directory and pip install -r requirements.txt is behaving like there is a clashing dependency graph. When it finishes locally, I'll try to figure out what the trigger dependency is.

lesamouraipourpre commented 3 years ago

The problem is adafruit-circuitpython-gizmo. There are requirements changes in it that have been merged but not released - https://github.com/adafruit/Adafruit_CircuitPython_Gizmo/compare/1.3.2...HEAD I had the HEAD installed with pip install -e . instead of the PyPi version.