Fix renaming of the ND_normalmap_vector2 node definition.

The upgrade code for the <normalmap> shader was indiscriminately
renaming all node definitions to "ND_normalmap_float". This included
"ND_normalmap_vecto2" nodes, whose nodedef should not be renamed.

This commit fixes that, limiting the renaming only to "ND_normalmap",
based on the type of the "scale" input, which is the only
differentiating factor between the two definitions.

source/MaterialXCore/Version.cpp

@@ -1316,8 +1316,17 @@ void Document::upgradeVersion()
             }
             else if (nodeCategory == "normalmap")
             {
-                // ND_normalmap was renamed to ND_normalmap_float
-                node->setNodeDefString("ND_normalmap_float");
+                // ND_normalmap was renamed to ND_normalmap_float.
+                // But, there is also ND_normalmap_vector2 that was not renamed.
+                // The only difference between them is the "scale" input type
+                // (float and vector2, respectively), so use that as a test.
+                NodeDefPtr nodeDef = getShaderNodeDef(node);
+                InputPtr scaleInput = node->getInput("scale");
+                if ((nodeDef && nodeDef->getName() == "ND_normalmap") ||
+                    (scaleInput && scaleInput->getType() == "float"))
+                {
+                    node->setNodeDefString("ND_normalmap_float");
+                }
 
                 node->removeInput("space");