Skip to content

remove unnecessary wait_for_frame call for CP5.0 compatibility #14

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 2 commits into from
Sep 3, 2019
Merged

remove unnecessary wait_for_frame call for CP5.0 compatibility #14

merged 2 commits into from
Sep 3, 2019

Conversation

jerryneedell
Copy link
Contributor

displayio no longer supports "wait_for_frame()" uses display.auto_refresh which is enabled by default.
use try/except for backward compatibility.

tested on feather_m4_express with TFT featherwing and on PyPortal

@jerryneedell jerryneedell requested a review from a team August 30, 2019 20:51
@makermelissa
Copy link
Collaborator

Looks good. Which versions of CP did you test with?

@jerryneedell
Copy link
Contributor Author

jerryneedell commented Aug 31, 2019

CP5.0 alpha and a build from prior to merge of the displayio changes.
I did not note the exact build -- I can retest with a known build if needed.

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

I don't think wait_for_frame is needed with 4.x either. Can you try it without? Thanks!

@jerryneedell
Copy link
Contributor Author

jerryneedell commented Sep 3, 2019

@tannewt correct - it was not needed in 4.x -- simply remove call to wait_for_frame

@jerryneedell jerryneedell changed the title add try/except for wait_for_frame call for CP5.0 compatibility remove unnecessary wait_for_frame call for CP5.0 compatibility Sep 3, 2019
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Perfect! Thank you!

@tannewt tannewt merged commit d01a230 into adafruit:master Sep 3, 2019
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Sep 6, 2019
Updating https://github.com/adafruit/Adafruit_CircuitPython_CursorControl to 2.1.3 from 2.1.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_CursorControl#12 from brentru/fix-pybadge

Updating https://github.com/adafruit/Adafruit_CircuitPython_Slideshow to 1.0.5 from 1.0.4:
  > Merge pull request adafruit/Adafruit_CircuitPython_Slideshow#14 from jerryneedell/jerryn_refresh

Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA:
  > Added the following libraries: Adafruit_CircuitPython_IL0373, Adafruit_CircuitPython_IL0398, Adafruit_CircuitPython_IL91874, Adafruit_CircuitPython_SSD1608, Adafruit_CircuitPython_SSD1675
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