Skip to content

refactor: src 구조 정리 및 패키지 통합 - #13

Closed
seunghyun0108 wants to merge 9 commits into
mainfrom
test/seunghyun
Closed

refactor: src 구조 정리 및 패키지 통합#13
seunghyun0108 wants to merge 9 commits into
mainfrom
test/seunghyun

Conversation

@seunghyun0108

Copy link
Copy Markdown
Collaborator

Summary

  • scout_ws 제거 및 패키지 구조 정리, bringup_pkg 통합

Changed files

  • scout_ws/ 제거, 패키지 src/ 바로 아래로 이동
  • 외부 패키지 third_party/로 분리
  • scout_mini_bringup → bringup_pkg 통합 및 이름 통일
  • launch 파일 패키지명 참조 수정

Test

  • colcon build
  • basic runtime check

Notes

  • launch 명령어 scout_mini_bringup → bringup_pkg 로 변경됨

Leejieun-0112 and others added 8 commits April 29, 2026 14:37
* a파일 생성함.

* add: CODEOWNERS
* a파일 생성함.

* main 수정

* restore docs from main

* 4.29 테스트분 모두 삭제
* 3번 노드 기능 + 5번 노드 기능 + bringup 폴더 추가

* 통합 프로토타입

* 통합 프로토타입 업데이트
@seunghyun0108
seunghyun0108 requested a review from SuBamm88 as a code owner May 29, 2026 17:27

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

return;
}

updateWithMax(master_grid, min_i, min_j, max_i, max_j);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Clear expired CCTV obstacles from the master costmap

When a pose disappears or exceeds max_observation_age_, updateBounds() resets this layer back to NO_INFORMATION and includes the previous bounds, but updateWithMax() does not write those unknown cells into the master grid. In the local costmap configuration, there is no static layer later overwriting those cells, so any CCTV obstacle that was once marked can remain lethal in the master costmap after the PoseArray is cleared or the observation ages out, causing navigation to avoid stale objects indefinitely.

Useful? React with 👍 / 👎.

@seunghyun0108
seunghyun0108 deleted the test/seunghyun branch May 29, 2026 19:02
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.

5 participants