[x3d-public] missing textureProperties field in ComposedCubeMapTexture, elsewhere?

[Michalis Kamburelis]

Indeed, textureProperties should be added to ComposedCubeMapTexture.
(Looking at CGE implementation, we support it already.) This is not
really related to PBR or my other changes,
"ComposedCubeMapTexture.textureProperties" always made sense in X3D
3.x too, it specifies e.g. filtering.

Actually it should be added to the ancestor, X3DEnvironmentTextureNode.

Actually it should be added to the higher ancestor, X3DSingleTextureNode :)

So we just found another advantage of having abstract
X3DSingleTextureNode. Everything descending from X3DSingleTextureNode
(which means: all textures except MultiTexture) deserves a
textureProperties field.

Regards,
Michalis

wt., 15 gru 2020 o 01:47 Don Brutzman <brutzman at nps.edu> napisał(a):
>
> Looking for additional opinions.  Michalis, can you please confirm (following your refactoring of all textures for PBR and NPR) that textureProperties should be added as described?  Also wondering if it is missing anywhere else.  TIA for all scrutiny.
>
> -----------------
> * Mantis 805: 34.4.1 ComposedCubeMapTexture - Missing textureProperties field
>    https://www.web3d.org/member-only/mantis/view.php?id=805
>
> -----------------
> Description: ComposedCubeMapTexture missing textureProperties field, may belong in X3DEnvironmentTextureNode
>
> ComposedCubeMapTexture missing textureProperties field.
>
> Note that textureProperties does appear in the other X3DEnvironmentTextureNode nodes, GeneratedCubeMapTexture and ImageCubeMapTexture.
>
> Probably textureProperties should simply be part of the common abstract type, X3DEnvironmentTextureNode.
>
> Reference:
> http://web3d.org/mailman/private/x3d_web3d.org/2015-May/003024.html
> -----------------
>
> all the best, Don
> --
> Don Brutzman  Naval Postgraduate School, Code USW/Br       brutzman at nps.edu
> Watkins 270,  MOVES Institute, Monterey CA 93943-5000 USA   +1.831.656.2149
> X3D graphics, virtual worlds, navy robotics http://faculty.nps.edu/brutzman