chenlin/Mujoco_WASM
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't add an empty "user" attribute to user sensors.

Browse Source 
This fixes the broken TAP tests.

PiperOrigin-RevId: 495867491
Change-Id: I740651ed0dd8d1c84053a48f3fca577b6024bf95
This commit is contained in:
Tom Erez
2022-12-16 07:16:55 -08:00
committed by Copybara-Service
parent f70403d00a
commit 189349f421
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
unity/Runtime/Components/Sensors/MjUserSensor.cs
+4 -2
View File
@@ -32,8 +32,10 @@ public class MjUserSensor : MjBaseSensor {
var mjcf = doc.CreateElement("user"); var mjcf = doc.CreateElement("user");
mjcf.SetAttribute("name", Name); mjcf.SetAttribute("name", Name);
mjcf.SetAttribute("dim", $"{Dimension}"); mjcf.SetAttribute("dim", $"{Dimension}");
// TODO: add validation that UserData is a space-separated list of floating numbers? if (!String.IsNullOrEmpty(UserData)) {
mjcf.SetAttribute("user", UserData); // TODO: add validation that UserData is a space-separated list of floating numbers?
mjcf.SetAttribute("user", UserData);
}
return mjcf; return mjcf;
} }