-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Closed as not planned
Labels
status: closed as inactiveIssues closed by the "Stale" Github Action. Please comment on any you think should still be open.Issues closed by the "Stale" Github Action. Please comment on any you think should still be open.status: inactiveMarked by the “Stale” Github ActionMarked by the “Stale” Github Action
Description
I've decided to review my work merging traitlets into matplotlib #4762, and during my rebase I came across this change to Artist.update
...
https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/artist.py#L880
where update
now handles values returned by setters.
I'm wondering what the motivation is behind these setters with return values, because they are incompatible with traitlets
. If traitlets
integration is to continue, these returning setters will need to be deprecated.
cc: @tacaswell, @jenshnielsen, @efiring
Metadata
Metadata
Assignees
Labels
status: closed as inactiveIssues closed by the "Stale" Github Action. Please comment on any you think should still be open.Issues closed by the "Stale" Github Action. Please comment on any you think should still be open.status: inactiveMarked by the “Stale” Github ActionMarked by the “Stale” Github Action