Defer SciPy and SymPy imports to cut `import spatialmath` time (~800 ms → ~150 ms)
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 68/100
- Type d'issue
- Refactorisation
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- python
- Domaine
- performance, testing
Piste de recherche
Start by reading the top-level imports and mentioned methods in base/quaternions.py, spline.py, base/symbolic.py, and base/argcheck.py. Compare the import-time regression approach from #198, then verify that import spatialmath leaves neither scipy nor sympy in sys.modules while the affected feature and SymPy-supported paths still work.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Follow-up to #198, which deferred matplotlib.pyplot until something actually plots. Its description noted that scipy.interpolate and sympy cost about as much. They are now the main remaining costs of import spatialmath.
Where the time goes
Measured with python -X importtime -c "import spatialmath" on Python 3.12, macOS, Apple M1 (8 GB), warm cache. Cold-start times will be higher.
| Module | Cost | Imported by |
|---|---|---|
scipy.interpolate + scipy.spatial.transform |
~455 ms | base/quaternions.py:19, spline.py:14-15 (both at top level) |
sympy (when installed) |
~200 ms | base/symbolic.py:19, via base/argcheck.py |
matplotlib (no pyplot) |
~100 ms | geom2d.py:13 from matplotlib.path import Path. #198 kept this on purpose because Polygon2 needs it for its geometry |
numpy |
~40 ms | |
| Total | ~800 ms |
SciPy: nothing needs it at import time
base/quaternions.py: the only user isqrand(). When the angle is limited, it does inverse-CDF sampling through_generate_inv_cdf_sin_squared_interp(), which is already@lru_cache'd. Every user pays ~440 ms for this one option.qslerpand the rest of the module don't use SciPy. Fix: move the import into that function, and make the-> interpolate.interp1dannotation a string or put it underTYPE_CHECKING.spline.py:CubicSpline,RotationSpline,RotationandBSplineare only used inside methods. Fix: import them inside those methods.- Both must change. If only one does, the other still loads
scipy.interpolate. SciPy stays a hard dependency; only when it loads changes. - #129 (open) adds more top-level SciPy imports to
spline.py(scipy.optimize,splrep) and would need the same treatment.
SymPy: avoid importing it just to check types
SymPy is optional. base/symbolic.py imports it inside a try so that issymbol() can check for sympy.Expr. But if "sympy" not in sys.modules, no argument can possibly be a SymPy expression. So issymbol() can return False without importing SymPy, and only import it once the user has imported SymPy themselves. This is a separate change from the SciPy one because argcheck is on almost every code path, so it needs its own testing (including the :SymPy: supported paths).
Expected result
About 150 ms for import spatialmath, and none of SciPy or SymPy loaded until a feature actually uses them. A regression test along the lines of #198's could assert that neither scipy nor sympy is in sys.modules after import spatialmath.
- Langage dominant
- Python
- Étoiles
- 643
- Forks
- 102
- Merge moyen
- 11 min
- PR mergées (30 j)
- 1
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de rai-opensource/spatialmath-python
-
documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
rai-opensource/spatialmath-python#235 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
rai-opensource/spatialmath-python#32 · 4 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 66/100
rai-opensource/spatialmath-python#236 ·
Les mainteneurs répondent en général sous 1 jour
-
Line3.isparallel()/__or__ still scale-dependent, like the distance() bug fixed in #224Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
rai-opensource/spatialmath-python#231 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 50/100
rai-opensource/spatialmath-python#177 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de rai-opensource/spatialmath-python
Issues similaires
-
needs-human needs-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
gke-labs/kube-agents#2400 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Device Details tables: FS/SF columns contradict each other (nfet_01v8 Vt row, pfet_01v8 Idsat row)Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
google/skywater-pdk#450 ·
-
Drained trajectory arrays are overwritten when the sequence buffer is reusedPeut-être pris @sylvesterkaczmarek l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
google-deepmind/bsuite#56 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
LearningCircuit/local-deep-research#7206 ·
Les mainteneurs répondent en général sous 1 jour
-
[TASK] Document technology stackOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
chingu-voyages/V62-tier3-team-33#285 ·
Les mainteneurs répondent en général sous 1 jour