Add Rotation and Typing

[DJDevon3]

Added new feature: ability to rotate display 0,90,180,270.

This will be particularly useful to arrange multiple charlieplexed matrixes together in any orientation for chaining. I don't think chaining multiple into 1 matrix is currently supported but it's a step in the right direction.

Added 16x9 charlieplex example that includes the new rotation feature.

"attempted" to add typing annotations... expect some to be wrong and that's ok, an attempt was made. i did not attempt to type every file. want to be sure i can do at least 1 correctly first then will do the other examples.

Rotation = 0 (hard to photograph because it's a scrolling marquee)

Rotation = 90

[FoamyGuy]

Thanks for working on this.

You're on the right track with many of the types but there are a few changes that can be made. Most of these are the same few things like Optional and return value tweaks but there are a few other sprinkled in as well.

[DJDevon3]

Oof, that'll keep me busy. I didn't try to type init on purpose because I thought you couldn't type init's.

Thank you for the review and the hints. I'm on it.

In the mean time here's a nice demo of the rotation.

2024-04-02_05-07-09.mp4

[DJDevon3]

Note to self don't put pound symbols in commit titles. github formatted it to point at adafruit issue number 2. i don't know how to fix that.

oh dear lord the amount of errors... pylint hates me.

[DJDevon3]

🏳️ I Surrender. Pylint has defeated me.

I defined ReadableBuffer in both typing and circuitpython_typing. Pylint won't recognize either of them.

[DJDevon3]

I can only conclude that ReadableBuffer does not exist in circuitpython_typing.

[DJDevon3]

There are now 4 ReadableBuffer errors instead of 1... So it was correctly setting 3 of them previously.

[DJDevon3]

I set it back and now there's only 1 ReadableBuffer error again...

[DJDevon3]

So I replaced the error on line 119
def _i2c_read_reg(self, reg: Optional[int] = None, result: Optional[ReadableBuffer] = None):
with
def _i2c_read_reg(self, reg: Optional[int] = None, result: Optional[int] = None):
and that passes but now errors on line 128 which is the next Optional[ReadableBuffer] use.

[DJDevon3]

Nope, I'm not allowed to use ReadableBuffer or WriteableBuffer in any fashion.

[DJDevon3]

[FoamyGuy]

The latest version of this is looking good to me.

Thank you for working through all of those changes @DJDevon3! I know it was quite specific and perhaps not the funnest coding task. I appreciate your work.

I took another look through everything in this branch and made 1 commit with a few additional tweaks for things I didn't see the first time through.

I tested the lastest version successfully on a Feather RP2040 9.0.4 with a CharlieWing using a modified simpletest to set some different pixels on and confirm behavior of the new rotation argument.

[DJDevon3]

@FoamyGuy God bless you. Thank you. 🙏 Yes that was a struggle. I don't want to see pylint again until next year.