Skip to content

Implement data inheritance in ImageBase._set_data()#13952

Merged
mattpap merged 3 commits intobranch-3.5from
mattpap/13951_ImageBase_inherited_data
Jun 27, 2024
Merged

Implement data inheritance in ImageBase._set_data()#13952
mattpap merged 3 commits intobranch-3.5from
mattpap/13951_ImageBase_inherited_data

Conversation

@mattpap
Copy link
Contributor

@mattpap mattpap commented Jun 27, 2024

This implements data inheritance in Image-like glyphs (Image, ImageRGBA and ImageStack). This was overlooked in PR #13554 and any previous follow-ups to that PR.

I suppose this could be backported to 3.4, though 3.4.2 was just released, so we would have to do that as a part of 3.4.3.

fixes #13951

@mattpap mattpap added this to the 3.5 milestone Jun 27, 2024
Copy link
Contributor

@philippjfr philippjfr left a comment

Choose a reason for hiding this comment

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

Looks great, thank you!

@bryevdv
Copy link
Member

bryevdv commented Jun 27, 2024

I would be +1 to release a 3.4.3 with this fix

@mattpap mattpap merged commit 2e34a08 into branch-3.5 Jun 27, 2024
@mattpap mattpap deleted the mattpap/13951_ImageBase_inherited_data branch June 27, 2024 23:00
mattpap added a commit that referenced this pull request Jun 27, 2024
* Implement data inheritance in ImageBase._set_data()

* Rename {_width,_height} -> image_{width,height}

* Add unit regression tests
@mattpap mattpap mentioned this pull request Jun 27, 2024
15 tasks
@mattpap mattpap modified the milestones: 3.5, 3.4.3 Jun 27, 2024
mattpap added a commit that referenced this pull request Jul 24, 2024
* Implement data inheritance in ImageBase._set_data()

* Rename {_width,_height} -> image_{width,height}

* Add unit regression tests
Chiemezuo pushed a commit to Chiemezuo/bokeh that referenced this pull request Aug 27, 2024
* Implement data inheritance in ImageBase._set_data()

* Rename {_width,_height} -> image_{width,height}

* Add unit regression tests
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Image Glyph causing tons of unnecessary calls to set_data

3 participants