Skip to content

Smoothing without scipy#3483

Merged
kaalleen merged 4 commits into
mainfrom
kaalleen/smoothing-without-scipy
Mar 2, 2025
Merged

Smoothing without scipy#3483
kaalleen merged 4 commits into
mainfrom
kaalleen/smoothing-without-scipy

Conversation

@kaalleen

@kaalleen kaalleen commented Feb 4, 2025

Copy link
Copy Markdown
Member

With these changes we can finally stop packaging scipy.

@kaalleen kaalleen force-pushed the kaalleen/smoothing-without-scipy branch from dbe7eb1 to 2f46a93 Compare February 5, 2025 21:41
@bkmgit

bkmgit commented Feb 9, 2025

Copy link
Copy Markdown
Collaborator

It maybe better to use an interpolation algorithim rather than dropping points.

@bkmgit

bkmgit commented Feb 9, 2025

Copy link
Copy Markdown
Collaborator

Not sure if https://numpy.org/doc/stable/reference/routines.polynomials-package.html#module-numpy.polynomial would work. For piecewise smooth curves, cubic splines might be a good choice, though these would likely add other dependencies as developing good software for these can take time.

@kaalleen

kaalleen commented Feb 9, 2025

Copy link
Copy Markdown
Member Author

This method is very simple, but works surprisingly well for the few use cases we have though.
But of course I am open for other smoothing routines. I just want to get rid of scipy as this is a bit much to include it only for smoothing. To use as few dependencies as possible will definitely make our lives easier once the extension installation routine for inkscape has been updated.

@kaalleen kaalleen merged commit 70d2ea5 into main Mar 2, 2025
@kaalleen kaalleen deleted the kaalleen/smoothing-without-scipy branch March 2, 2025 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants