Track a dirty area for in-memory bitmaps by tannewt · Pull Request #1999 · adafruit/circuitpython · GitHub
Skip to content

Track a dirty area for in-memory bitmaps#1999

Merged
dhalbert merged 3 commits into
adafruit:4.1.xfrom
tannewt:turtleboosters
Jul 19, 2019
Merged

Track a dirty area for in-memory bitmaps#1999
dhalbert merged 3 commits into
adafruit:4.1.xfrom
tannewt:turtleboosters

Conversation

@tannewt

@tannewt tannewt commented Jul 19, 2019

Copy link
Copy Markdown
Member

This fixes the bug that bitmap changes do not cause screen updates
and optimizes the refresh when the bitmap is simply shown on the
screen. If the bitmap is used in tiles, then changing it will
cause all TileGrids using it to do a full refresh.

Fixes #1981

This fixes the bug that bitmap changes do not cause screen updates
and optimizes the refresh when the bitmap is simply shown on the
screen. If the bitmap is used in tiles, then changing it will
cause all TileGrids using it to do a full refresh.

Fixes micropython#1981
@tannewt tannewt added this to the 4.1.0 milestone Jul 19, 2019
@tannewt tannewt requested a review from dhalbert July 19, 2019 00:02

@dhalbert dhalbert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just one question about the algorithm.

Comment thread shared-module/displayio/Bitmap.c
Comment thread shared-module/displayio/TileGrid.c Outdated
@dhalbert

Copy link
Copy Markdown
Collaborator

@tannewt

tannewt commented Jul 19, 2019

Copy link
Copy Markdown
Member Author

Ok, @dhalbert. It should be ready for re-review and pass Travis 🤞 .

@dhalbert dhalbert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Another fantastic speedup!

@dhalbert dhalbert merged commit dbab538 into adafruit:4.1.x Jul 19, 2019
@dhalbert

Copy link
Copy Markdown
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants