Johannes Günther

What about orientation? It seemed we largely agree that it is important to have https://github.com/KhronosGroup/glTF/pull/1850#issuecomment-747720622, yet 2b2a17420d15cd023d4011ef2b395552298610ef removes `rotation`. A rotation is easy to apply for cubemap lookups and also...

There are multiple questions here: 1. Allow rotation or not? It's useful to tweak the look and illumination, and it would allow for a dynamic (animated, rotating) environment. The alternative...

> I don't really understand the usecase here - could you please elaborate? Conceptually I see the environment as fixed and non rotatable, just like it is in the real...

I was thinking just of the HDRI environment texture, to be placed as a light at nodes. An implementation / renderer could derive the SH coefficients or other pre-filtering from...

Any feedback/discussion/review?

The main purpose of the bilinear patch is the demonstration of how to write a module, a plugin to extend OSPRay. We test its compilation to catch that this module...

In the superbuild the bilinear patch module is activated via `BUILD_OSPRAY_CI_EXTRAS`.

Yes, as said, the module servers as example how to write modules. The bilinear patch geometry is not indented as first class geometry. Is this rather a feature request, i.e.,...

To install your own module you could use `ospray_install_library`, see e.g. `modules/multiDevice/CMakeLists.txt`. We are happy to improve the dev process based on your suggestions. Please note that when writing custom...

Before the `OSPFuture` we indeed had a "progress-and-cancel" callback. There are some upcoming chances to the futures in the mid-term, we'll consider your input.