fix: Transformation nodes must create new scope #20

glen · 2024-02-14T08:09:41Z

glen commented

2024-02-14 08:09:41 +00:00

Improve the handling of transformation nodes so that their translations always open a new scope.
As a side effect, expands handling of transformation nodes to include Scale and Transform as well.

Improve the handling of transformation nodes so that their translations always open a new scope. As a side effect, expands handling of transformation nodes to include Scale and Transform as well.

glen added 2 commits 2024-02-14 08:09:42 +00:00

b6c5c6d0a5 fix: Transformation nodes must create new scope

In VRML 1, transformations like translations and rotations
  accumulate within a scope as they are encountered, and one
  can even have multiple translations with shapes interleaved
  between them. This accumulation is not possible in VRML97.
  Therefore, it's necessary to insert new scopes in the translated
  file every time a new transformation is encountered. This
  commit makes sure that happens.

b5b4065d6b doc: Prior changes mean all transformation nodes supported

glen merged commit 5bd62ffe31 into main

2024-02-14 08:10:01 +00:00

glen deleted branch apply_transform

2024-02-14 08:10:01 +00:00

glen referenced this issue from a commit

2024-02-14 08:10:02 +00:00

fix: Transformation nodes must create new scope (#20)

Sign in to join this conversation.

No reviewers