Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolve ambiguity around RPC glTF importing #39

Open
jwnimmer-tri opened this issue May 16, 2023 · 1 comment
Open

Resolve ambiguity around RPC glTF importing #39

jwnimmer-tri opened this issue May 16, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@jwnimmer-tri
Copy link
Contributor

We apply a 90 degree rotation after importing the glTF we got over RPC. It is not sufficiently clear why we do this.

Is Blender importing the file incorrectly? Or is its importer documentation unclear about applying extra rotations that we need to undo? Or is Drake exporting the scene incorrectly? It has to be one or those, so we need figure out which one and (at least) write it down. If it's Drake doing it wrong, we'll also need to fix that.

Sean's comment from review:

        bpy.ops.transform.rotate(
            value=math.pi/2, orient_axis='X', orient_type='GLOBAL'
        )

Does this feel like a defect in VTK's glTF exporter? glTF is documented as y-up. Blender is z-up. If we need to rotate positive 90 degrees around the x-axis, it's because blender applied, correctly, the negative 90 degree rotation to go back from y-up to z-up. Admittedly, as long as the server knows about the idiosyncracies of the client's glTF flavor, they can form a coherent whole. But it does mean we're producing non-compliant glTF files.

@jwnimmer-tri
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant