`scipy.spatial.transform.Rotation` Integration
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia con le chiamate riprodotte a scipy.spatial.transform.Rotation.apply e ispeziona le interfacce di VectorObject3D e VectorNumpy3D. Confronta gli input diretti di Rotation con gli input di liste di array funzionanti, quindi definisci il completamento come l’accettazione di istanze di Rotation per entrambe le forme di vettore, con le rotazioni mostrate che producono i risultati attesi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Vector Version
1.1.1.post1
Python Version
3.10.9
OS / Environment
OS: KAli Linux 2023.3
Environment: scipy==1.11.1
Describe the bug
First of all thanks for the awesome work on the library
there are some errors with Rotation.apply
these are some ways to reproduce
from scipy.spatial.transform import Rotation
from vector import VectorObject3D, VectorNumpy3D
v_object = VectorObject3D(x=1, y=0, z=0)
v_array = VectorNumpy3D({"x": [1], "y": [0], "z": [0]})
r = Rotation.from_quat((0, 0, 1, 1))
# _r = r.as_quat()
# print(v.rotate_quaternion(_r[3], _r[0], _r[1], _r[2]))
# Returns: VectorObject3D(x=0.0, y=0.9999999999999998, z=0.0)
# print(r.apply(v_object)) # Raises: IndexError(tuple index out of range)
print(r.apply(v_object.__array__().tolist()))
# Returns: [0. 1. 0.]
# print(r.apply(v_array)) # Raises: ValueError(Expected input of shape (3,) or (P, 3), got (1,).)
print(r.apply(v_array.tolist()))
# Returns: [[0. 1. 0.]]
I think it would be a good idea to add rotate method to Vector that accepts the Rotation instance as input and applies the rotation on Vector
from scipy.spatial.transform import Rotation
def rotate(self, rot: Rotation):
pass
Any additional but relevant log output
No response
- Lingua principale
- Python
- Stelle
- 101
- Fork
- 43
- Merge medio
- 3h 38m
- PR unite (30g)
- 7
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di scikit-hep/vector
-
Numba object backend: missing to_2D/to_3D/to_4D, momentum-named conversions, and like()Forse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Apertafeature
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
scikit-hep/vector#750 ·
-
Add ability to somehow map field names to coordinate axis names that vector expectsForse di nuovo libera @hjaat l’ha presa 81 giorni fa e non c’è nessuna pull request aperta. Apertafeature
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
scikit-hep/vector#732 · 3 commenti ·
-
feature
Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100
scikit-hep/vector#731 · 1 commento ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
scikit-hep/vector#711 · 1 commento ·
-
Pyright showing errorsApertabug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
scikit-hep/vector#654 ·
Tutte le issue di scikit-hep/vector
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
jbacalso24/mimry#291 ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 82/100
eli-alvarado/us-watchlist-alerts#14 ·
I maintainer di solito rispondono entro 1 giorno
-
from-review-extraction python severity:nit
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
LearningCircuit/local-deep-research#7462 ·
I maintainer di solito rispondono entro 2 giorni
-
mermaid_extract.py silently discards single-line state-diagram notes; sequence notes are keptAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
cathrynlavery/diagram-design#441 ·
I maintainer di solito rispondono entro 2 giorni
-
[Cleanup] Include the public header in work_split.cppForse già presa @blozano-tt l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
tenstorrent/tt-metal#60850 ·
I maintainer di solito rispondono entro 1 giorno