Skip to content

User can mark components as wheels [SYNTH-198] - #1455

Open
BrandonPacewic wants to merge 26 commits into
devfrom
branp/198/user-select-add-joints
Open

User can mark components as wheels [SYNTH-198]#1455
BrandonPacewic wants to merge 26 commits into
devfrom
branp/198/user-select-add-joints

Conversation

@BrandonPacewic

Copy link
Copy Markdown
Member

Task

SYNTH-198

Symptom

If a user did not have a cad model with properly jointed wheels they could not drive or use their robot in Synthesis without first making potentially non-trivial changes to their CAD.

Solution

Allow for the user to select components on their robot and mark them as wheels after the import step.

Note

This currently only works for arcade drive.

Verification

  • Import a robot that does not have wheels already assigned, say the 2026 Kitbot via URDF.
  • It won't drive on the initial import.
  • Select the wheels of the kitbot, be sure to select the outer most component of each wheel.
  • Wait for the assembly to be rebuilt, then drive!

Before merging, ensure the following criteria are met:

  • All acceptance criteria outlined in the ticket are met.
  • Necessary test cases have been added and updated.
  • A feature toggle or safe disable path has been added (if applicable).
  • User-facing polish:
    • Ask: "Is this ready-looking?"
  • Cross-linking between Jira and GitHub:
    • PR links to the relevant Jira issue.
    • Jira ticket has a comment referencing this PR.

* origin/branp/171/urdf-import-support: (37 commits)
  feat: missing material fallback
  fix: memory management stuff
  chore: update submodules
  chore: don't specify autodesk registry for jolt-physics
  chore: update joltjs
  Robots Float Downwards Within the Move Tool `[SYNTH-173]` (#1383)
  Remove TransformAssemblyPanel.tsx `[SYNTH-212]` (#1382)
  refactor: minor naming
  fix: lint
  refactor: move isWheel to separate file
  fix: axis alignment bug
  refactor: move other functions to `ConstraintSettingsUtilities.ts`
  refactor: move constraint functinos to separate file
  chore: some commenting and renaming
  refactor: ball constraints
  fix: hinge limits
  fix: slider constraints
  chore: cleanup imports
  refactor: standardize interface for applying limits
  chore: remove non-null asserts
  ...
…add-joints

* origin/dev: (72 commits)
  fix: `.npmrc` ignore
  fix: make assetpack tests actually fail on problems
  chore: move ignore to top level dir
  chore: global dot ignore
  fix: remove npmrc for github actions
  chore: fix ci failures
  feat: top level mira assembly urdf tagging
  update in more places
  Make Zone Config Panel Pretty `[SYNTH-225]` (#1402)
  Fix Mirabuf Loader Tests
  Merge Assetpack Properly
  feat: optimize obj parser
  fix: axis joints for wheel baised on urdf imports
  feat: add gltf parser
  fix: re-render scoring zones after resetting preferences
  Fix Double Sliders on Manage Protected Zones Interface `[SYNTH-164]` (#1396)
  format
  fix: refactor review comments & enable obj support
  move divider
  revert to dim green
  ...
* 'dev' of github.com:Autodesk/synthesis: (171 commits)
  fix: better comment pattern
  Don't Pause Audio on Mobile Chromium `[SYNTH-240]` (#1442)
  Hotfix scoreboard initial state `[SYNTH-276]` (#1445)
  feat: strip template comments and truncate pr body
  fix: file comments in a review no longer send twice
  biome: warn on useComponentExportOnlyModules
  biome: elevate naming lint to error
  multiplayer/types: resolve warnings from __
  RobotDimensionsTracker: resolving lint warnings
  format: fix arrow function formatting
  ThemeProvider: resolve updating themeOptions warning
  resolve more naming issues
  UICallbacks: ignore __ function lints
  components/CommandPalette: temporarily ignore registryTick lint
  chore(deps): bump actions/upload-artifact from 4 to 7
  chore(deps): bump actions/setup-python from 5 to 7
  fix: rename v4 to uuidv4
  feat: update progress bar incrementally during URDF part building
  fix: dismissals should now show dismiss message
  chore: removed some comments, reformatted
  ...
@BrandonPacewic BrandonPacewic added the gameplay Relating to the playability of Synthesis label Jul 28, 2026
BrandonPacewic and others added 3 commits August 4, 2026 10:25
* 'dev' of github.com:Autodesk/synthesis: (120 commits)
  fix: update array bracket formatting
  chore: refactor out new build parts functionality
  chore: update comments
  feat: obj mesh reduction fix
  feat: glTF dense meshes extra rung
  feat: mesh decimation for obj and gltf
  chore: update comments
  chore: cleanup comments
  feat: enable caching for urdf imports
  chore: mesh optimizer package
  feat: refactor mesh decimation
  feat: mesh decimation for stl urdf imports
  fix: reduce urdf import memory usage on kitbot
  chore: always use T[] for arrays (#1466)
  fix(config): drop the "Configuration saved" toast
  fix(topbar): splitdropdown sounds
  feat(topbar): toggle panel suggestion via topbar icons
  feat(UIProvider): toggle panel implementation
  fix(topbar): divider for code connection hidden
  fix: fallback to jolt mass if mass is 0
  ...
@rutmanz
rutmanz marked this pull request as ready for review August 5, 2026 22:58
@rutmanz
rutmanz requested review from a team as code owners August 5, 2026 22:58
@rutmanz
rutmanz requested a review from PepperLola August 5, 2026 22:58
@rutmanz
rutmanz force-pushed the branp/198/user-select-add-joints branch from 81135fc to 7b83c73 Compare August 12, 2026 23:52

@AlexD717 AlexD717 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uploading the given URDF Kitbot and assigning wheels results in the back of it floating up for me (as seen in the attached video).

Screen.Recording.2026-08-21.at.10.19.37.AM.mov

Almost the same floating issue happens on the UI selection for me (where I verified that I properly selected the wheels), though Ernie is properly assigned and the console errors don't appear.

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

Labels

gameplay Relating to the playability of Synthesis

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants