To correctly compute the center of mass of a path (to e.g. center a marker correctly), we need to correctly compute the path lengths of the component Bezier curves.

Roadmap:

#16812 (*) <-#16832 (*) <-#16859 (*) <- (This PR) (*) <-#16889 (*) <-#16891 (MarkerStyle improvements!)

"<-" means "depends on", and "(*)" marks PRs whose implementations are complete and fully ready for review.

PR Checklist

Has Pytest style unit tests
Code isFlake 8 compliant
New features are documented, with examples if plot related
Documentation is sphinx and numpydoc compliant
Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

This was referencedMar 23, 2020

Path center of mass#16889

Draft

Equal area markers#16891

Draft

Compute area of path#16859

Open

Correctly compute path extents#16832

Merged

Bezier/Path API Cleanup: fix circular import issue#16812

Merged

brunobeltran force-pushed thepath_length branch fromaca0710 to7c72a61Compare

March 23, 2020 23:27

QuLogic added status: work in progress status: waiting for other PR labels

Mar 24, 2020

brunobeltran force-pushed thepath_length branch 9 times, most recently frome2f403a to8d1e757Compare

March 31, 2020 02:03

brunobeltran force-pushed thepath_length branch 3 times, most recently from23ffc9f to0dbef92Compare

April 9, 2020 05:01

brunobeltran force-pushed thepath_length branch from0dbef92 tocaf0324Compare

April 20, 2020 03:05

jklymak marked this pull request as draft

July 23, 2020 03:17

github-actionsbot added the status: needs rebase label

Oct 18, 2022

Copy link

github-actionsbot commentedJul 14, 2023

Since this Pull Request has not been updated in 60 days, it has been marked "inactive." This does not mean that it will be closed, though it may be moved to a "Draft" state. This helps maintainers prioritize their reviewing efforts. You can pick the PR back up anytime - please ping us if you need a review or guidance to move the PR forward! If you do not plan on continuing the work, please let us know so that we can either find someone to take the PR over, or close it.

github-actionsbot added the status: inactiveMarked by the “Stale” Github Action label

Jul 14, 2023

Copy link

Member

jklymak commentedJul 14, 2023

@brunobeltran your work on these path PRs was great. Appreciate that you have a real job now, but if you have any time to finish them, it would be a real service. Maybe your company allows donated hours?

github-actionsbot removed the status: inactiveMarked by the “Stale” Github Action label

Jul 17, 2023

Copy link

github-actionsbot commentedSep 18, 2023

github-actionsbot added the status: inactiveMarked by the “Stale” Github Action label

Sep 18, 2023

Copy link

Contributor

greglucas commentedSep 5, 2024

@rossbar @ksunden - Here is the PR I wanted to link you to for the bezier path things you were asking about today at the NumFOCUS summit. I think if anyone wants to pick this work up it would be very welcome and it has been laid out really nicely in the PR descriptions.