Skip to content

fix: sensor init bug and clean up app READMEs#107

Open
xiaolefang-dm wants to merge 2 commits intomainfrom
xiaole/fix/sensor-init-and-readme
Open

fix: sensor init bug and clean up app READMEs#107
xiaolefang-dm wants to merge 2 commits intomainfrom
xiaole/fix/sensor-init-and-readme

Conversation

@xiaolefang-dm
Copy link
Copy Markdown
Contributor

Changes

Bug fix: _detect_and_init_sensor incomplete startup

  • Previously only launched perception_node (launched twice, first process leaked)
  • Now calls _launch_sensor_procs to start the full stack per mode:
    • looper: looper_bridge + planning
    • realsense: realsense driver + perception + planning

README cleanup

  • Remove references to non-existent run_web_app.sh
  • Use uv run python instead of python3 for consistency

…eb_app.sh from READMEs

- Fix _detect_and_init_sensor: was only starting perception (twice, with
  a leaked first process). Now calls _launch_sensor_procs to start the
  complete stack per mode (looper: bridge+planning, realsense: driver+perception+planning).
- Remove run_web_app.sh references from app/README.md and app/frontend/README.md.
@github-actions
Copy link
Copy Markdown

cloc github.com/AlDanial/cloc v 2.04 T=0.16 s (246.3 files/s, 36539.7 lines/s)
Language files blank comment code
Python 12 580 257 3234
Markdown 4 155 6 426
C++ 4 60 38 409
Bourne Shell 14 32 9 242
Dockerfile 1 30 29 191
JSON 2 8 0 58
TOML 1 4 0 57
make 1 18 15 55
CMake 1 4 0 17
-------- -------- -------- -------- --------
SUM: 40 891 354 4689

@xiaolefang-dm xiaolefang-dm requested a review from dvorak0 April 25, 2026 06:16
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.

1 participant