Skip to content

feat: UX improvements for web demo - #45

Merged
arjo129 merged 1 commit into
open-rmf:epic/next-gen-basefrom
arjo129:arjoc/feat/ux_changes
Jul 22, 2026
Merged

feat: UX improvements for web demo#45
arjo129 merged 1 commit into
open-rmf:epic/next-gen-basefrom
arjo129:arjoc/feat/ux_changes

Conversation

@arjo129

@arjo129 arjo129 commented Jul 22, 2026

Copy link
Copy Markdown
Member

Based on user testing it was found that:

  1. The reset button is flaky. For now, this is not a high priority thing to fix so we are disabling it.
  2. A tooltip hinting to click on a robot was added.

Note: When testing remember to disable cache.

GenAI Use

We follow OSRA's policy on GenAI tools

  • I used a GenAI tool in this PR.
  • I did not use GenAI

Generated-by: Gemini

Based on user testing it was found that:
1. The reset button is flaky. For now, this is not a high priority thing
   to fix so we are disabling it.
2. A tooltip hinting to click on a robot was added.

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>

@mxgrey mxgrey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These UI changes are really nice improvements 👍

@mxgrey

mxgrey commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

The CI gave a strange error:

--- stderr: rmf_path_server
error: failed to get `rstar` as a dependency of package `parry3d v0.21.1`
    ... which satisfies dependency `parry3d = "^0.21"` of package `mapf_post v0.1.0 (https://github.com/arjo129/mapf_post?branch=arjo%2Ffeat%2Fmore_efficient_checks#07a61faf)`
    ... which satisfies git dependency `mapf_post` of package `rmf_path_server v0.1.0 (/__w/next_gen_prototype/next_gen_prototype/next_gen_ws/src/next_gen_prototype/path_server/rmf_path_server)`

Caused by:
  failed to load source for dependency `rstar`

Caused by:
  unable to update registry `crates-io`

Caused by:
  download of rs/ta/rstar failed

Caused by:
  curl failed

Caused by:
  [16] Error in the HTTP2 framing layer
---

I'll have it rerun in case this is just an intermittent problem.

@arjo129

arjo129 commented Jul 22, 2026

Copy link
Copy Markdown
Member Author

The CI gave a strange error:

--- stderr: rmf_path_server
error: failed to get `rstar` as a dependency of package `parry3d v0.21.1`
    ... which satisfies dependency `parry3d = "^0.21"` of package `mapf_post v0.1.0 (https://github.com/arjo129/mapf_post?branch=arjo%2Ffeat%2Fmore_efficient_checks#07a61faf)`
    ... which satisfies git dependency `mapf_post` of package `rmf_path_server v0.1.0 (/__w/next_gen_prototype/next_gen_prototype/next_gen_ws/src/next_gen_prototype/path_server/rmf_path_server)`

Caused by:
  failed to load source for dependency `rstar`

Caused by:
  unable to update registry `crates-io`

Caused by:
  download of rs/ta/rstar failed

Caused by:
  curl failed

Caused by:
  [16] Error in the HTTP2 framing layer
---

I'll have it rerun in case this is just an intermittent problem.

Smells like network issues.

@arjo129
arjo129 merged commit a176984 into open-rmf:epic/next-gen-base Jul 22, 2026
5 of 6 checks passed
@github-project-automation github-project-automation Bot moved this from Inbox to Done in PMC Board Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants