Link.__deepcopy__ silently drops collision geometry (coal pickling limitation)
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 48/100
Rechercherichtung
Führe test_ERobot.py aus, gefolgt von test_Link.py::TestDHLink::test_copy mit --tb=long. Verfolge die Attributkette vom neuen DHLink zum coal CollisionObject über den gemeinsam genutzten Zustand von Link oder Robot. Erledigt bedeutet, dass die unbeabsichtigte gemeinsame Referenz entfernt oder als Weak Reference verwendet wird und der gezielte Copy-Test den Warnpfad ausführt, ohne nicht zugehörige Geometrie zu verlieren.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Migrated from tech-debt.md (deleted, see repo history via git log -- tech-debt.md).
Link.__deepcopy__ silently drops coal CollisionObject instances because the coal library doesn't support pickling. The current workaround warns at runtime when shapes are lost.
Root cause: a fresh DHLink can reach the coal objects of an unrelated URDF robot through shared class-level state in Link or Robot (exact attribute TBD). This means:
- Copied DH links that happen to run after URDF robot tests lose nothing (DH links have no collision shapes), so the warning path is never exercised in isolation.
- If the shared reference is ever followed for other purposes (iteration, serialisation) it could cause similar failures or unexpected aliasing.
Proper fix: obtain the full deepcopy traceback with --tb=long when running test_ERobot.py followed by test_Link.py::TestDHLink::test_copy, trace which attribute chain connects the fresh DHLink to a coal object, and remove or weak-ref that shared state.
- Vorherrschende Sprache
- C++
- Sterne
- 3.6k
- Forks
- 645
- Ø Merge
- 1 T. 8 Std.
- Gemergte PRs (30 T.)
- 145
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus petercorke/robotics-toolbox-python
-
plt.pause(0) waits for ever: PoseGraph.plot_scan(pause=0) and Vehicle animation may hangEvtl. vergeben @huxiaoyi-ovo hat das heute übernommen. Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
petercorke/robotics-toolbox-python#876 · 1 Kommentar · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
petercorke/robotics-toolbox-python#709 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Isaac Sim backendOffenenhancement
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
petercorke/robotics-toolbox-python#903 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Backend registry: single name → constructor dispatchEvtl. vergeben @petercorke hat das heute übernommen. Offenenhancement
petercorke/robotics-toolbox-python#902 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
tech-debt
Schwierigkeit 4/5 1-2 Tage Anfängerfreundlichkeit 55/100
petercorke/robotics-toolbox-python#898 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in petercorke/robotics-toolbox-python
Ähnliche Issues
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
-
bug code-review testing tooling
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
aristocratos/btop#1945 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 79/100
boykopovar/AnyPS5#11291 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
k2-fsa/sherpa-onnx#4100 ·
Maintainer antworten meist innerhalb von 2 Tagen