Fix animation rigged referenced scenes transform issue (#417)

GPUOpen-LibrariesAndSDKs · Nov 14, 2023 · e0adc43 · e0adc43
1 parent bd2a9e5
commit e0adc43
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 2 deletions.
diff --git a/FireRender.Maya.Src/Context/FireRenderContext.h b/FireRender.Maya.Src/Context/FireRenderContext.h
@@ -563,6 +563,10 @@ class FireRenderContext : public IFireRenderContextInfo
 	{
 		assert(!uuid.empty());
 
+#ifdef _DEBUG
+		std::string uuidNameWithoutInstanceIndex = FireRenderObject::uuidWithoutInstanceNumberForString(uuid);
+#endif
+
 		auto it = m_mainMeshesDictionary.find(FireRenderObject::uuidWithoutInstanceNumberForString(uuid));
 
 		if (it != m_mainMeshesDictionary.end())

diff --git a/FireRender.Maya.Src/FireRenderObjects.cpp b/FireRender.Maya.Src/FireRenderObjects.cpp
@@ -99,7 +99,16 @@ std::string FireRenderObject::uuidWithoutInstanceNumber() const
 
 std::string FireRenderObject::uuidWithoutInstanceNumberForString(const std::string& uuid)
 {
-	return splitString<std::string>(uuid, ':')[0];
+	size_t pos = uuid.find_last_of(":");
+	if (pos == std::string::npos)
+		return uuid;
+
+	if (!isNumber<std::string>(uuid.substr(pos + 1, std::string::npos)))
+		return uuid;
+
+	std::string ret = uuid.substr(0, pos);
+
+	return ret;
 }
 
 void FireRenderObject::setDirty()

diff --git a/FireRender.Maya.Src/FireRenderUtils.h b/FireRender.Maya.Src/FireRenderUtils.h
@@ -1259,6 +1259,12 @@ std::vector<T> splitString(const T& s, typename T::traits_type::char_type delim)
 	return elems;
 }
 
+template <class T>
+bool isNumber(const std::string& s) 
+{
+	return !s.empty() && std::all_of(s.begin(), s.end(), ::isdigit);
+}
+
 // Backdoor to enable different AOVs from Render Settings in IPR and Viewport
 void EnableAOVsFromRSIfEnvVarSet(FireRenderContext& context, FireRenderAOVs& aovs);
 

diff --git a/FireRender.Maya.Src/frWrap.h b/FireRender.Maya.Src/frWrap.h
@@ -3787,7 +3787,15 @@ namespace frw
 		{
 			rpr_int res = rprMaterialNodeSetID(Handle(), id);
 
-			assert(res == MStatus::kSuccess);
+			if (res == RPR_ERROR_UNSUPPORTED ||
+				res == RPR_ERROR_INVALID_PARAMETER)
+			{
+				// should not cancel execution if such error happens
+			}
+			else
+			{
+				checkStatus(res);
+			}
 		}
 
 	};