cjio icon indicating copy to clipboard operation
cjio copied to clipboard

cjio exports inaccurate when translation is large

Open wlinna opened this issue 9 months ago • 1 comments

Describe the bug cjio exports tend to become very inaccurate once translation is involded.

My suspicion is that it is caused by cjio calling self.decompress before computing normals, triangulating etc. The problem would likely go away if only the scale was applied before computations, and then the translation was applied as a final step. In the case of glTF, the translation should be set for the node. In the case of OBJ, the translation should be applied to each vertex.

cjio version I tried both the master and develop branches

To Reproduce Prerequisite: you have cjio in your PATH

repro.zip

  1. Download the test files
  2. Run cjio simplified.city.json export glb simplified.glb
  3. Run cjio simplified_translated.city.json export glb simplified_translated.glb
  4. Open Blender. Import simplified.glb . It looks okay. You can remove it
  5. Import simplified_translated.glb
  6. Select the node called "building", press F3, sand run "Origin to Geometry"
  7. Set the location to 0, 0, 0
  8. See how deformed the GLB is

I also tested OBJ, and the problem is basically the same. (The reason why we must take so many steps to view the file is because the translation is applied to vertex coordinates instead of setting it for the node)

Expected behavior The resulting geometry should not deform.

(Also, as a side note, the GLB should have the translation applied to its root node, not at geometry level. The viewers have much easier time handling such models)

Screenshots SimplifiedTranslated

Desktop (please complete the following information):

  • OS: Pop!_OS (Ubuntu-based)
  • Version 22.04

Additional context This happens with real-life models. This is just a simplified version with made-up coordinates, but the problem is the same.

wlinna avatar May 04 '24 18:05 wlinna