EXT_mesh_primitive_edge_visibility

[pmconne]

The EXT_mesh_primitive_edge_visibility extension augments a triangle mesh primitive with sufficient information to enable engines to produce non-photorealistic visualizations of 3D objects with visible edges. The edge visibility is encoded in a highly compact form to avoid excessively bloating the glTF asset. Adapted for glTF from iTwin/itwinjs-core#5581. We intend to add support for the extension to the CesiumJS engine.

[pjcozzi]

@pmconne so excited for this!

• @lilleyse could you please take a quick look from a 3D Tiles and CesiumJS perspective?
• @kring could you take a quick look from a Cesium Native and game engine perspective?

[kring]

Thanks for opening this @pmconne! I have a few thoughts below.

Overall it looks good! I can't see any particular reason it couldn't be implemented in Unreal Engine or Unity.

[pmconne]

@kring @javagl @pjcozzi thank you for your feedback. I've revised the spec accordingly.

[pjcozzi]

@kring @javagl @lexaknyazev or others - do you have any additional feedback for @pmconne on this extension?

[kring]

@pmconne I read through the spec again and I think this looks great! No further suggestions from me.

I would like to see a complete implementation of this extension in CesiumJS or iTwinJS, though, if there isn't one already. It would be useful both as proof that it can be implemented sensibly in WebGL (2?), and for my own interest in the rendering techniques involved.

[pmconne]

I would like to see a complete implementation of this extension in CesiumJS or iTwinJS

Most of the rendering pieces are already in place in iTwin.js. I think the quickest path to a complete implementation would be for me to update the iModel -> 3D Tiles pipeline to encode the edges according to the spec and update iTwin.js decode accordingly.

[weegeekps]

@lexaknyazev Any other changes required preventing us from getting this approved and merged? Like EXT_primitive_restart, this is intended to just be an open multi-vendor extension.

[markschlosseratbentley]

The spec has been updated to now list CesiumJS as a known implementation in addition to iTwin.js. These implementations have the caveat that they do not implement the full capability of this specification, namely materials. This support is forthcoming.

[CLAassistant]

All committers have signed the CLA.

[lexaknyazev]

In addition to the review comments please optimize all PNG images with

$ oxipng --zopfli --strip all -o max *.png

[danielzhong]

@lexaknyazev Thanks! I’ve gone through and updated all the PR feedback.