Skip to content

Conversation

@HunterBarclay
Copy link
Member

@HunterBarclay HunterBarclay commented Nov 27, 2025

Task

AARD-2113

Julian broke my wiring panel. I fix

Problem

Styling issues since UI refactor for the wiring panel.

image

Verification

Screenshot 2025-11-26 at 11 18 23 PM Screenshot 2025-11-26 at 11 18 06 PM

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.

Copy link
Member

@PepperLola PepperLola left a comment

Choose a reason for hiding this comment

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

When I run the regular Java simulation sample and load Dozer, it doesn't immediately display the simulation inputs and outputs. I can refresh the Robot IO panel and that will sometimes make them load properly. Initially it took multiple refreshes to get it to work but I haven't been able to reproduce that since. The refresh/reset button at the top removes all the simulation inputs and outputs and makes another Robot IO panel refresh necessary.

image

When I add a junction node it shows no inputs or outputs until I refresh from the Robot IO panel.

image

@HunterBarclay HunterBarclay changed the title fix: Wiring Panel fix: [AARD-2113] Wiring Panel Styling Fixes Nov 27, 2025
@HunterBarclay
Copy link
Member Author

@PepperLola Odd. Java is borked on my laptop right now and I haven't been able to test fully with robot code yet. Let me know if you can see any issues appearing in the console.

const [nodes, setNodes, onNodesChange] = useNodesState([] as FlowNode[])
const [edges, setEdges, onEdgesChange] = useEdgesState([] as FlowEdge[])
const [_refreshHook, refreshGraph] = useReducer(x => !x, false) // Whenever I use reducers, it's always sketch. -Hunter
const [refreshHook, refreshGraph] = useReducer(x => !x, false) // Whenever I use reducers, it's always sketch. -Hunter
Copy link
Member Author

Choose a reason for hiding this comment

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

@PepperLola I've been using this as a janky way to refresh the graph. Probably need to re-evaluate.

!: Auto Tester doesn't work and it appears the robot can't be controlled
via robot code at the moment...
@HunterBarclay HunterBarclay requested a review from a team as a code owner December 1, 2025 08:54
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.

3 participants