More transformations of shapes
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Following up on #1489, it seems like using .transform() or .transformed() in any subclass of compas.geometry.Shape will result in the scale component of the transformation to be ignored, because only the shape's frame is transformed.
There's a commented out piece of code in Box that seems to have attempted a fix to that, doing matrix decomposition of the transformation and then applying the scale to the dimensions of the box, but for some reason is commented out and it's also not implemented in any of the other shapes (as far as I checked).
As a first step, we should write unit tests that verify the transform and transformed results for each of the shapes in compas.geometry (sub-classes of Shape). If these tests fail, it means the transformation is indeed not applying the scale component, so the second step would be to add the implementation for that, and hopefully get the unit tests to pass.
- Lingua principale
- Python
- Stelle
- 387
- Fork
- 124
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
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 compas-dev/compas
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
compas-dev/compas#1523 · 3 commenti ·
-
Numpy and Rhino 9Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
compas-dev/compas#1507 · 2 commenti · 1 reazione ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
compas-dev/compas#1504 · 1 commento ·
-
`Polygon.normal` with `normal_polygon` instead of `normal_triangle`?Forse già presa @copilot-swe-agent l’ha presa 351 giorni fa. Aperta
compas-dev/compas#1487 · 1 reazione · 1 assegnatario ·
-
Support for unit and uncertainty (tolerance)Forse già presa @copilot-swe-agent l’ha presa 386 giorni fa. Aperta
compas-dev/compas#1482 · 1 reazione · 1 assegnatario ·
Tutte le issue di compas-dev/compas
Issue simili
-
Path subtraction removes repeated suffix characters instead of one suffixForse già presa @Gwerdonatus l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
joke2k/django-environ#671 ·
-
good first issue help wanted windows
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
-
test_pgbouncer deploys the server under test from Charmhub instead of the local build (track/1.31)Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
canonical/temporal-k8s-operator#195 · 1 commento ·
I maintainer di solito rispondono entro 4 giorni
-
`burr` fails to start when the Python path has a space in itForse già presa @enekoreto l’ha presa oggi. Apertastatus/needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 5 giorni
-
device support request
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
zigpy/zha-device-handlers#5461 ·
I maintainer di solito rispondono entro 4 giorni