Skip to content

fix(vnc): increase VNC and XServer timeouts for macOS/ARM64#266

Open
Dev-sh21 wants to merge 2 commits intoJdeRobot:humble-develfrom
Dev-sh21:fix/macos-vnc-timeout
Open

fix(vnc): increase VNC and XServer timeouts for macOS/ARM64#266
Dev-sh21 wants to merge 2 commits intoJdeRobot:humble-develfrom
Dev-sh21:fix/macos-vnc-timeout

Conversation

@Dev-sh21
Copy link

hi @jmplaza @javizqh
this pr address the stability issue, preventing the robotics acadamy from starting on mac os with apple silicon (m1,m2,m3),

for all these devices, the docker container runs under QEMU Emulation which introduces significant performance overhead around 10 times slower than naive x86 and the current hardcoded timeouts are insufficnent for this environment.

As the result the application continously crashes with TimeoutError, before the simulation can finish loading
Screenshot 2026-02-13 at 6 31 08 PM

to solve this, i have increased the timeout to accomodate the emulation overload

  1. vnc_server.py --- increased to 120 sec
  2. process_utils.py---- increased by 120 sec

this works reliabely on macos ans has no negative impact on native Linux users.

Copy link
Member

@javizqh javizqh left a comment

Choose a reason for hiding this comment

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

Only missing a inline documentation change

@Dev-sh21 Dev-sh21 force-pushed the fix/macos-vnc-timeout branch from e5ae0ad to 0a0610c Compare February 14, 2026 08:29
super().__init__()
self.update_callback = callback
self.server = WebsocketServer(port=port, host="127.0.0.1")
self.server = WebsocketServer(port=port, host="0.0.0.0")
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this

Copy link
Author

Choose a reason for hiding this comment

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

sir, i have reverted the host bindings to 127.0.0.1, the other changes are in place, please check it once

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