Merging 0.28.5 into dev-adsk (AnimalLogic#9)

* MayaReference: shuffle code so it only tries to edit ns on loaded refs

* fix import of AL.usd.maya.schemas

* fix TranslatorContext::updatePrimTypes crash

When updatePrimTypes was removing prims from the m_primMapping (should
it even do that?), it wasn't using std::vector::erase correctly.

* Fix for when tearing down a shape that it's sibling shapes are left alone

* TranslatePrim now correctly works with mergedTransform flag

* Move renderer selection to LayerManager

* Introduce RendererManage command

* Introduce code style changes

* Make m_rendererPlugin a string attribute

* Add documentation concerning renderer selection

* Revert NodeHelper changes

* Introduce to LayerManager rendererPlugin internal attribute and rendererPluginName storable attribute

* Default to EarliestTime timecode

* Added "forceDefaultRead" option for import and translate

This is mainly to handle usd meshes which have generated with
one time sample attribute values instead of using the default
value.

* Default to EarliestTime timecode

* [AL_USDMayaUtils] Updated nurbs utils API

* Ensure that time is always connected to imported proxy nodes - also don't rename transform nodes we have no right in renaming

* Updated tests cmake configuration

* [AL_USDMaya] Updated forceDefaultRead import parameter's doc

* BugFix where you could only push changes to a Mesh once through the TranslateMesh Prim

* [schemas] Fixed Windows exports

* glext.h is not in the devkit anymore, switching files

* Fixes for the Windows build :
- Adding boost chrono lib only for Windows builds
- Adding Python libraries dependency to fix link error on compile
- Changing the cmake lookup file for plugins as glext.h will not be in a
future devkit.

* extract useful functions into AL_USDMayaUtils

* remove unnecessary using

* fix windows build

* geometry diffing

* fixed up some windows build errors & warnings

fixed some more compiler warnings & errors for clang & VC

more compiler warning fixes

fixed maya2018 deprecation warning

more build warning fixes Win32/clang

fixed issue with prims not removing the parent transforms

signed/unsigned mistmatch

* unregister managerenderer

* version up for utils package update

* make LayerDatabase automatically hide non-dirty layers

* remove old edit target from layerManager if it wasn't dirty

* LayerDatabase now no longer has an easily queryable size, so add bool testing

* Renamed functions

* Fixed unittests regarding the LayerManager, it's lifetime and dirty tracked layers

* LayerManager Command

* Minor for AL_USDMayaUtils because of helper function refactor

* Updated to 0.27.13

* Update README.md

* Update README.md

* Construct excluded prims list on attribute change

* Consider excludedTranslatedGeometry changes

* Move renderer related code to LayerManagerRenderer.cpp

* Introduce RendererManager node

* Introduce suggestions

* Fixed InternalProxyShapeSelect to be used on selection change event

* Removed debug saving of a Maya file in a test

* removed the code that deletes files from /tmp, it's not needed

* ensure the time value from the options is passed through to the export options.

* removed the /tmp paths so the tests can run on Win32

* Added documentation for the maya mesh data

* removed docs on leftHanded geometry, and fixed some grammar

* fixed up doxygen warnings/errors. Removed a few incorrectly exported functions (they are inline). Removed duplicate setFloatArray/setDoubleArray

* removed the leftHanded mesh handling code - the same result can be achieved with the opposite attribute

* Skipped schemas tests if needed

* [schemas] API schema as a non-applied schema

* [schemas] Generated schema source code

* Updated build doc

* [translators] Fixed tests environment

* Updated CHANGELOG

* Boost date time temporarily added to Windows build

* Updated patch version

* Updated doc for LayerManager

* make getRenderAttris update showRender

* fixes to allow modification of rpath via command line

ie, with this change, you can pass this to cmake:

-D CMAKE_INSTALL_RPATH=/path/to/usd/lib/dir

...to make it so that you don't need to modify $LD_LIBRARY_PATH in order for the
plugin to load / link against USD.  If you do this, I also recommend passing
these to cmake:

-D CMAKE_EXE_LINKER_FLAGS=-Wl,--enable-new-dtags
-D CMAKE_SHARED_LINKER_FLAGS=-Wl,--enable-new-dtags

...as this will make the resultant binary use RUNPATH, not RPATH, which is
overridable by LD_LIBRARY_PATH.

* add libAL_USDMayaTranslators's dir to it's RPATH, so it can find libAL_USDMayaSchemas

* update the viewport for progressive renderers (like embree)

* ProxyShapeUI: add trailing newline to TF_DEBUG statements

* Updated cmake files (removed unused)

* fixed issue with inverted normals

* 0.28.2 update

* added a command to create a prim on a usdstage of a specific type & kind

* moved shape naming code to prior the fileIO kicking in

* make outStageData connectable

* pass correct path to the selection

* removed unused include

* fixed a number of build wanrings on the OSS linux build

* fixed version.h, and updated changelog.md

* fix for regression wrt to transform values not being correctly reported into maya

* added additional selection events for USDGlimpse to listen to

* prevent changes in transform values being written to usd if the value has not changed

* only time attribute to transforms that are actually transforms

* also disconnect stage to non-valid TM's

* make sure loadStage is called when a proxyShape is duplicated, so that
the new shape is ready for use.

match braces style

* allowed certain event types (those with Text Name ID) to have callbacks registered against them when the event type has yet to be registered

* remove pointless exports from maya event handler, and fix up message type names

* Update build.md

* Update faq.md

* Warnings as errors (Windows build)

* python docs

* ProxyShape docs

* fixed up build warnings in opensource build

* address robs comments

* Updated to 0.28.4

* Add ALFrameRange openSource schema

* Add ALFrameRange openSource translator

* Add ALFrameRange translator tests

* Add ALFrameRange tutorial

* Fix the issue that translator decref parent transform chain too agressively

* Add test for the fix for over-decref transform chain issue

* Manually bump up patch version in Version.h

* Update change log

CHANGELOG.md

@@ -1,3 +1,99 @@
+## v0.28.5 (2018-07-10)
+
+### Added
+
+* New open-source ALFrameRange schema and translator, which sets up Maya time range and current frame during the translation.
+* Tests for ALFrameRange translator.
+* Tests for the fix for the over-decref issue in transform chain.
+
+### Fixed
+
+* Over-decref in transform chain during some prim tear-down.
+
+## v0.28.4 (2018-07-05)
+
+### Added
+
+* New events to prevent USDGlimpse from rebuilding the scene when mapping meta data is inserted into the prims during selection.
+
+### Changed
+
+* Transform values that have not changed no longer get written back to USD.
+* Only hook up the proxy shape time attribute to the transform nodes that are actually transforms (i.e. ignore prims that are effectively just organisational containers)
+* Allowed named events to be able to have callbacks registered against them prior to the events being registered
+
+### Removed
+
+* Some export macros from inline methods
+
+### Fixed
+
+* Regression where transform values were no longer being correctly displayed when translating prims into maya.
+* A number of build warnings in the OSS build
+* Final two build warnings in the repo. The generated schema code needs to have a flag set in the build C4099 to hide the 'class defined as struct' warning (pixar issue, not ours)
+* When you duplicate a proxy shape, you wont see anything in the viewport and it wont be fully initialize until you reopen maya or find a hack (like changing the usd path) to trigger a load stage. (#98 - @nxkb)
+* Add missing newlines to some TF_DEBUG statements in ProxyShapeUI (#92 - @elrond79)
+
+## v0.28.3 (2018-06-18)
+
+### Added
++ AL_usdmaya_CreateUsdPrim command added to insert a new prim into the UsdStage of a proxy shape. 
+
+### Changed
++ The proxyShape outStageData is now connectable, and now longer hidden. Allows for manual DG node connections to be made. 
+
+### Fixed
++ The internal mapping between a maya object and a prim now works correctly when specifying the -name flag of AL_usdmaya_ProxyShapeImport. 
+
+## v0.28.2 (2018-06-14)
+
+### Added
++ Viewport will continue to refresh to show updated progressive renderers (like embree) until converged (#91 - @elrond79)
+
+### Changed
++ make getRenderAttris update showRender, as well as showGuides (#87 - @elrond79)
+
+### Fixed
++ Tweaks to allow builds to work with just rpath. (#88 - @elrond79)
++ When importing left handed geometry that has no normals, compute the correct set of inverted normals.
+
+## v0.28.1 (2018-06-06)
+
+### Added
+* Documentation for mesh export, interpolation modes, and diffing.
+* "-fd" flag added to the import command and the translate command
+* Support for prim var interpolation modes in the import/export code paths (currently UV sets only, colour sets & normals will be in a later PR).
+* Support for diffing all variations of the interpolation modes on prim vars.
+* Routines to determine the interpolation modes on vec2/vec3/vec4 types.
+-  AL_usdmaya_LayerManagerCommands to retrieve layers that have been modified and that have been the EditTarget.
+
+### Changed
++ Built using USD-0.8.5
++ ProxyShapePostLoadProcess::createTranformChainsForSchemaPrims now checks all the prims' parents' metadata for their unmerged status. If found, the transform chain creation skips the current prim in place of its parent.
++ ProxyShapePostLoadProcess::createSchemaPrims now checks all the prims' parents' metadata for their unmerged status. If found, the schema prim import will instead take place under the parent node.
++ Mesh::import now checks the prim's parents' metadata for its unmerged status. If found, the created shape node will not have the appended "Shape" string.
+* Mesh attributes queries use EarliestTime timecode
+* Functions in DgNodeTranslator are moved to DgNodeHelper in AL_USDMayaUtils
+* AttributeType.h/.cpp are moved to AL_USDMayaUtils
+- LayerManager API has been updated so you can now inspect all layers that have been set as an EditTarget and are Dirty.
++ Updated endToEndMaya tutorial to work with latest USD API changes.
+* All hard coded /tmp paths in the unit tests, in favour of a linux/win32 solution.
+
+### Fixed
+
+- Fix for when tearing down a shape, that it's sibling shapes are left alone in Maya.
+* time1.outTime is now connected to proxy shape nodes when transforms are selected during creation.
+* Selected transforms are no longer renamed when importing a proxy shape.
+* Tests build restored.
+- Fix for Issue "meshTransform cannot execute twice" #82
+* Bug in the import/export of mesh normals
+* Normals export now correctly listen to the exporter options.
+* AL_usdmaya_plugin can be unloaded correctly.
+* Passing the time value into the nurbs, camera, and transform translators on export.
+
+### Removed
+* Removed the code that reversed polygon windings if the 'leftHanded' meta data flag was encountered. Instead, the 'opposite' flag is set on the maya mesh.
+
 ## v0.27.10 (2018-05-07)
 
 ### Added

CMakeLists.txt

@@ -18,8 +18,7 @@ include(${USD_CONFIG_FILE})
 # to get PYTHON_EXECUTABLE
 find_package(PythonInterp)
 
-include(ProjectDefaults)
-include(Public)
+set(CMAKE_INCLUDE_CURRENT_DIR ON)
 include(CXXDefaults)
 
 add_definitions(${_PXR_CXX_DEFINITIONS})
@@ -64,6 +63,7 @@ set(NEED_BOOST_FILESYSTEM ON)
 if(WIN32)
     set(CMAKE_CXX_FLAGS
         -std=c++11
+        /WX
         "${_PXR_CXX_FLAGS}"
     )
 else()
@@ -80,6 +80,11 @@ set(CMAKE_VERBOSE_MAKEFILE OFF)
 option(BUILD_USDMAYA_SCHEMAS "Build optional schemas." ON)
 option(BUILD_USDMAYA_TRANSLATORS "Build optional translators." ON)
 option(VALIDATE_GENERATED_SCHEMAS "Generate schemas code and compare." OFF)
+option(SKIP_USDMAYA_TESTS "Build tests" OFF)
+
+if(NOT SKIP_USDMAYA_TESTS)
+    enable_testing()
+endif()
 
 set(AL_USDMAYA_LOCATION_NAME
     "AL_USDMAYA_LOCATION"

README.md

@@ -10,11 +10,17 @@ Pixar's [USDMaya plugin](https://graphics.pixar.com/usd/docs/Maya-USD-Plugins.ht
 ## Contact
 We have a google group for discussion and technical assistance: https://groups.google.com/forum/#!forum/al_usdmaya-discussion. For CLAs or to contact us directly, use email  **[email protected]**.
 
+## Videos
+We have some videos explaining some of the workflows we support. We are hoping to add to this as we go:
++ [Animation & Layout](https://youtu.be/wcC71Yu2gjA)
++ [Modelling & Surfacing](https://youtu.be/DaxLk6pHijw)
+
 
 ## Detailed Documentation
 
 + [Building it](docs/build.md)
 + [FAQ](docs/faq.md)
++ [Python](docs/python.md)
 + [Contributing](docs/contributing.md)
 + [Translation](docs/translation.md)
 + [Basic Usage](docs/basicUsage.md)

cmake/defaults/CXXDefaults.cmake

@@ -48,6 +48,11 @@ if(CMAKE_BUILD_TYPE STREQUAL "Debug")
     _add_define(BUILD_OPTLEVEL_DEV)
 endif()
 
+# Set plugin path environment variable name
+if (PXR_OVERRIDE_PLUGINPATH_NAME)
+    _add_define("PXR_PLUGINPATH_NAME=${PXR_OVERRIDE_PLUGINPATH_NAME}")
+endif()
+
 set(_PXR_CXX_FLAGS ${_PXR_CXX_FLAGS} ${_PXR_CXX_WARNING_FLAGS})
 
 # CMake list to string.

cmake/defaults/Options.cmake

@@ -58,6 +58,12 @@ set(PXR_INSTALL_LOCATION ""
     "Intended final location for plugin resource files."
 )
 
+set(PXR_OVERRIDE_PLUGINPATH_NAME ""
+    CACHE
+    STRING
+    "Name of the environment variable that will be used to get plugin paths."
+)
+
 set(PXR_ALL_LIBS ""
     CACHE
     INTERNAL

cmake/defaults/Version.cmake

@@ -24,4 +24,4 @@
 # Versioning information
 set(PXR_MAJOR_VERSION "0")
 set(PXR_MINOR_VERSION "8")
-set(PXR_PATCH_VERSION "3")
+set(PXR_PATCH_VERSION "5")

cmake/defaults/msvcdefaults.cmake

@@ -26,8 +26,8 @@
 set(_PXR_CXX_FLAGS "${_PXR_CXX_FLAGS} /EHsc")
 
 # Standards compliant.
-set(_PXR_CXX_FLAGS "${_PXR_CXX_FLAGS} /Zc:rvalueCast    \
-                                      /Zc:strictStrings \
+set(_PXR_CXX_FLAGS "${_PXR_CXX_FLAGS} /Zc:rvalueCast
+                                      /Zc:strictStrings
                                       /Zc:inline")
 
 # Turn on all but informational warnings.