Skip to content
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

Update HX8357 driver to 8.x/9.x transitional code #23

Merged
merged 3 commits into from
Mar 25, 2024

Conversation

DJDevon3
Copy link
Contributor

@DJDevon3 DJDevon3 commented Mar 24, 2024

3.5" TFT Featherwing display driver needs updating for 9.0. Added 8.x to 9.x transitional code so it should work in either 8.x or 9.x Only tested in 9.x, needs 8.x testing. In 9.x the previous driver commit would bring up future warnings.

FutureWarning: Display moved from displayio to busdisplay
FutureWarning: Display renamed BusDisplay

These are now addressed with this commit and no warnings are present while using 9.0

@dhalbert dhalbert requested a review from a team March 24, 2024 04:00
"""HX8357D driver"""

def __init__(self, bus: displayio.FourWire, **kwargs) -> None:
def __init__(self, bus: FourWire.FourWire, **kwargs) -> None:
Copy link
Member

Choose a reason for hiding this comment

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

The type annotation now just be FourWire, but this is great, thanks!

Copy link
Member

@tekktrik tekktrik left a comment

Choose a reason for hiding this comment

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

One small change, but otherwise, this is great! Good find and fix.

@DJDevon3
Copy link
Contributor Author

DJDevon3 commented Mar 25, 2024

Thanks! I helped someone in discord with this exact issue last night and never dawned on me to check the typing I set here. Doh! Also got Github Desktop sorted out. It's submitting all line endings in LF format only. No more thousands of pylint CRLF line ending errors.

Copy link
Member

@tekktrik tekktrik 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 great, thank you!

@tekktrik tekktrik merged commit 11dddce into adafruit:main Mar 25, 2024
1 check passed
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Mar 26, 2024
Updating https://github.com/adafruit/Adafruit_CircuitPython_DisplayIO_SH1107 to 1.6.2 from 1.6.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_DisplayIO_SH1107#20 from FoamyGuy/display_root_group_fix

Updating https://github.com/adafruit/Adafruit_CircuitPython_HX8357 to 1.4.2 from 1.4.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_HX8357#23 from DJDevon3/main

Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Text to 3.1.0 from 3.0.6:
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Text#203 from FoamyGuy/outlined_label

Updating https://github.com/adafruit/Adafruit_CircuitPython_FeatherWing to 1.15.2 from 1.15.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_FeatherWing#82 from tekktrik/dev/change-arg-name

Updating https://github.com/adafruit/Adafruit_CircuitPython_RSA to 1.2.21 from 1.2.20:
  > Merge pull request adafruit/Adafruit_CircuitPython_RSA#41 from FoamyGuy/json_key_file_example

Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA:
  > Updated download stats for the libraries
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