From 8d64c63f492c568c31b001c3e986039bd38c9359 Mon Sep 17 00:00:00 2001 From: bd_ Date: Mon, 15 Apr 2024 11:43:16 +0900 Subject: [PATCH] fix: disable parameter cost check for now (#821) This creates compatibility issues with VRCF, as well as being run in the wrong phase of processing in any case. At some point this will need to be restored, as part of a new NDMF processing phase that runs much later in the build. Fixes: #813, #806 --- Editor/RenameParametersHook.cs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Editor/RenameParametersHook.cs b/Editor/RenameParametersHook.cs index 561f4f6b..697633b7 100644 --- a/Editor/RenameParametersHook.cs +++ b/Editor/RenameParametersHook.cs @@ -260,7 +260,8 @@ namespace nadena.dev.modular_avatar.core.editor } expParams.parameters = parameters.ToArray(); - + + /* if (expParams.CalcTotalCost() > VRCExpressionParameters.MAX_PARAMETER_COST) { BuildReport.LogFatal("error.rename_params.too_many_synced_params", new[] @@ -270,6 +271,7 @@ namespace nadena.dev.modular_avatar.core.editor } ); } + */ avatar.expressionParameters = expParams; }