drwave / usd-sketchup

Source & macOS binary of USD exporter for SketchUp
Other
63 stars 5 forks source link

We should report faces as well as meshes in the summary panel #22

Closed drwave closed 5 years ago

drwave commented 5 years ago

For those that will care.

drwave commented 5 years ago

We also might want to track the original number of vertices along with the number we export (since we triangulate everything).

drwave commented 5 years ago

So this is a little complicated. Initially I thought we'd track the original number of faces, and then the resulting number of triangles.

But there are two things that make this more complicated:

So what would we want this to look like? We currently put out the number of Meshes, but I realize now that is just in component definition and non-component instance uses - we aren't tracking the number of actually meshes that end up in the scene with the instances.

So probably want to rethink the way we're tracking all this info, unfortunately...

drwave commented 5 years ago

So most of this work is done and checked in, and seems to work as expected.

One last thing before I close this out, though - need to add shader counts.

drwave commented 5 years ago

We report it all, and we report it well.