Skip to content

Remove max_size parameter #32

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 1 commit into from
Jul 2, 2021

Conversation

lesamouraipourpre
Copy link
Contributor

Remove the max_size parameter. It is no longer used by displayio.Group
Ref: adafruit/circuitpython#4959

Examples tested on a Pynt and Blinka where possible:

Adafruit CircuitPython 6.3.0 on 2021-06-01; Adafruit PyPortal with samd51j20

The MatrixPortal and MagTag examples were not tested as I don't have the hardware.

Copy link
Contributor

@jposada202020 jposada202020 left a comment

Choose a reason for hiding this comment

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

I tested on

Adafruit CircuitPython 6.3.0 on 2021-06-01; Adafruit MagTag with ESP32S2

with the
progressbar_magtag_simpletest.py

Results

Adafruit CircuitPython 6.3.0 on 2021-06-01; Adafruit MagTag with ESP32S2
>>> import progressbar_magtag_simpletest
92.167
95.167
98.167
95.167
92.167
89.167

LGTM

@jposada202020 jposada202020 merged commit 6b5e569 into adafruit:main Jul 2, 2021
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Jul 7, 2021
Updating https://github.com/adafruit/Adafruit_CircuitPython_CLUE to 2.3.0 from 2.2.9:
  > Merge pull request adafruit/Adafruit_CircuitPython_CLUE#46 from jposada202020/adding_note
  > Merge pull request adafruit/Adafruit_CircuitPython_CLUE#45 from lesamouraipourpre/max-size
  > Moved default branch to main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template
  > "Increase duplicate code check threshold "

Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Button to 1.6.0 from 1.5.5:
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Button#32 from lesamouraipourpre/requirements
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Button#31 from lesamouraipourpre/max-size
  > Moved default branch to main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_ProgressBar to 2.3.0 from 2.2.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_ProgressBar#32 from lesamouraipourpre/max-size
  > Moved default branch to main
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