search

SimulaVR / Simula

Linux VR Desktop
MIT License
2.95k stars 87 forks source link

Moving the Weston node elsewhere in the scene tree crashes #43

Closed lboklin closed 5 years ago

lboklin commented 6 years ago

Specifically it seems to depend on the relative node path between the Weston and the ARVROrigin nodes. Some assumption is made somewhere regarding the layout of the scene tree hierarchy.

Possibly related (or even the cause) is the code in Weston.hs that looks for the controller nodes with hard-coded paths. In general I think assumptions about node paths should rarely, if ever, be made anywhere in the code. Also, the code pertaining to other nodes in Weston.hs should perhaps be deprecated (at the cost of two-handed resizing) or at least moved and implemented in some other way.

lboklin commented 5 years ago

Needs to be reproduced after recent refactors.

lboklin commented 5 years ago

Can't reproduce with the Simula class (which is our new "main").