Skip to content

ENH: VRMLImporter/Exporter improper handling of ambient and specular

Bill Lorensen requested to merge lorensen/vtk:lorensen/vtk-VRMLImporter/ExporterHandlingAmient into master

The exporter was doing several things incorrectly.

  1. Creating a directional light by default. This is removed.
  2. Only setting emissive color if there are no normals or no scalar colors or no polygons. This restriction is removed. All actor properties will define an emissive color.

The importer was doing several things incorrectly.

  1. The specular power needed a scale of 127 to convert shininess.
  2. The specular factor was not set.

Here is the bad export/import:

BadVRML

versus the fixed export/import:

GoodVRML

Edited by Bill Lorensen

Merge request reports