Skip to content

Conversation

@ychaparov
Copy link

@ychaparov ychaparov commented Sep 28, 2023

Proposed change(s)

In the same Unity project, we use URDF-Importer in Unity Editor, but we also have scenes that should be built for WebGL.

As noted in #201 WebGL builds of a project that has URDF-Importer package always fail, even if URDF-Importer is not used during runtime.

This commit marks the conflicting plugins to exclude WebGL.
NOTE: URDF-Importer scripts will not be usable in WebGL builds: Runtime/Unity.Robotics.URDFImporter.asmdef still excludes WebGL. This change allows scenes that don't use URDF-Importer to build.

Useful links (GitHub issues, JIRA tickets, forum threads, etc.)

#201

Types of change(s)

  • Bug fix
  • New feature
  • Code refactor
  • Documentation update
  • Other (please describe)

Testing and Verification

Build a scene that doesn't use URDF-Importer runtime components, while the Unity project has URDF-Importer package

Test Configuration:

  • Unity Version: 2022.3.4f1
  • Unity machine OS + version: Ubuntu 22.04

Checklist

  • Ensured this PR is up-to-date with the dev branch
  • Created this PR to target the dev branch
  • Followed the style guidelines as described in the Contribution Guidelines
  • Added tests that prove my fix is effective or that my feature works
  • Increased the test coverage criteria by 3%
  • Updated the Changelog and described changes in the Unreleased section
  • Updated the documentation as appropriate

Other comments

In the same project, we use URDF-Importer in Unitye Editor, but we also have scenes
that are built for WebGL.

As noted in Unity-Technologies#201
WebGL builds of a project that has URDF-Importer package always fail, even if
URDF-Importer is not used during runtime.

This commit marks the conflicting plugins to exclude WebGL.
NOTE: URDF-Importer scripts will not be usable in WebGL builds: Runtime/Unity.Robotics.URDFImporter.asmdef
still excludes WebGL. This change allows scenes that don't use URDF-Importer to build.
@unity-cla-assistant
Copy link

unity-cla-assistant commented Sep 28, 2023

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants