matplotlib / matplotlib/matplotlib

Merge event_handling/{path_editor,poly_editor} examples

Open
#17,314 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Documentation keep
Dominant language
Python
Stars
23.2k
Forks
8.5k
Avg merge
1d 6h
Merged PRs (30d)
66

Description

The two examples are very close to one another, except possibly for rather subtle differences(?) (https://github.com/matplotlib/matplotlib/pull/11431#issuecomment-623075067). They can probably be merged together.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Compare the event_handling/path_editor and event_handling/poly_editor examples, starting with the differences noted in the linked PR comment. Confirm whether their subtle behavioral differences matter before consolidating them. Done means the redundant examples are merged without losing the distinct behavior that the comparison identifies.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data-visualization, documentation
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.