Skip to content

Implement Tire Trajectory feature and cleanup Path Modes#342

Merged
ajouatom merged 2 commits into
ajouatom:c3-wipfrom
ChanHoya:c3-wip
May 4, 2026
Merged

Implement Tire Trajectory feature and cleanup Path Modes#342
ajouatom merged 2 commits into
ajouatom:c3-wipfrom
ChanHoya:c3-wip

Conversation

@ChanHoya
Copy link
Copy Markdown

@ChanHoya ChanHoya commented May 3, 2026

  1. 신규 파라미터 등록: CarrotTireTrajectory를 추가하여 타이어 경로 표시 여부를 독립적으로 제어할 수 있게 했습니다.
  2. 설정 메뉴 개편: ㅇ Carrot 설정의 Path 탭에서 Laneless Path 설정 바로 위에 "Tire Trajectory" 토글 메뉴를 추가했습니다. ㅇ 기존 패스 모드 선택(Laneless, LaneMode) 리스트에서 더 이상 필요 없는 16번 모드를 삭제하고 최대 범위를 15로 조정했습니다.
  3. 렌더링 로직 최적화: ㅇ drawLaneCenterIndicator 함수에서 불필요했던 중앙 패스 그리기 로직을 완전히 삭제했습니다. ㅇ 외곽 그라데이션(Tire Trajectory)과 하단 L/R 거리 표시는 설정 메뉴에서 해당 토글을 켰을 때만 나타나며, 기존에 선택한 Laneless 또는 LaneMode 패스와 동시에 깨끗하게 겹쳐서 보입니다.

@chanhojung
Implement Tire Trajectory feature and cleanup Path Modes
214cb1a 
1. 신규 파라미터 등록: CarrotTireTrajectory를 추가하여 타이어 경로 표시 여부를 독립적으로 제어할 수 있게 했습니다.
2. 설정 메뉴 개편:
ㅇ Carrot 설정의 Path 탭에서 Laneless Path 설정 바로 위에 "Tire Trajectory" 토글 메뉴를 추가했습니다.
ㅇ 기존 패스 모드 선택(Laneless, LaneMode) 리스트에서 더 이상 필요 없는 16번 모드를 삭제하고 최대 범위를 15로 조정했습니다.
3. 렌더링 로직 최적화:
ㅇ drawLaneCenterIndicator 함수에서 불필요했던 중앙 패스 그리기 로직을 완전히 삭제했습니다.
ㅇ 외곽 그라데이션(Tire Trajectory)과 하단 L/R 거리 표시는 설정 메뉴에서 해당 토글을 켰을 때만 나타나며, 기존에 선택한 Laneless 또는 LaneMode 패스와 동시에 깨끗하게 겹쳐서 보입니다.
@ChanHoya
Copy link
Copy Markdown
Author

ChanHoya commented May 3, 2026

지난번 제가 PR 했던 mode 16은 다른 모드 115와 중복해서 보여지고 있었네요.
그래서 mode 16을 없애고, 별도 TireTrajectory(타이어 궤적과 유사) 옵션으로 활성화 하게하고, experimental 모드의 path 효과를 가운데 넣은 것을 제거하였습니다.
따라서 mode 16을 활성화 하면 mode 115 는 병행하여 보여지는 구조로 변경하였습니다.
그런데 PR 결과가 오류가 나오는데, 저는 현재 작동이 잘되고 있습니다.

@ajouatom
Copy link
Copy Markdown
Owner

ajouatom commented May 3, 2026

carrot_settings.json 파일에도 추가바랍니다.

@ajouatom
Copy link
Copy Markdown
Owner

ajouatom commented May 3, 2026

drawLaneCenterIndicator 로 인해 기존 path color가 흐트러지는것 같습니다..
mode 16을 사용하지 않으려면 흔적도 삭제해야합니다.
약간의 코드를 수정합니다..

@ajouatom ajouatom merged commit 9508758 into ajouatom:c3-wip May 4, 2026
4 of 15 checks passed
ajouatom added a commit that referenced this pull request May 4, 2026
@ajouatom
Revert "Implement Tire Trajectory feature and cleanup Path Modes (#342)"
9aa4036 
This reverts commit 9508758.
happymaj11r pushed a commit to happymaj11r/openpilot that referenced this pull request May 4, 2026
@ChanHoya @chanhojung @ajouatom
Implement Tire Trajectory feature and cleanup Path Modes (ajouatom#342)
75d305a 
1. 신규 파라미터 등록: CarrotTireTrajectory를 추가하여 타이어 경로 표시 여부를 독립적으로 제어할 수 있게 했습니다.
2. 설정 메뉴 개편:
ㅇ Carrot 설정의 Path 탭에서 Laneless Path 설정 바로 위에 "Tire Trajectory" 토글 메뉴를 추가했습니다.
ㅇ 기존 패스 모드 선택(Laneless, LaneMode) 리스트에서 더 이상 필요 없는 16번 모드를 삭제하고 최대 범위를 15로 조정했습니다.
3. 렌더링 로직 최적화:
ㅇ drawLaneCenterIndicator 함수에서 불필요했던 중앙 패스 그리기 로직을 완전히 삭제했습니다.
ㅇ 외곽 그라데이션(Tire Trajectory)과 하단 L/R 거리 표시는 설정 메뉴에서 해당 토글을 켰을 때만 나타나며, 기존에 선택한 Laneless 또는 LaneMode 패스와 동시에 깨끗하게 겹쳐서 보입니다.

Co-authored-by: Hoya <jungch@naver.com>
Co-authored-by: carrot <43668841+ajouatom@users.noreply.github.com>
(cherry picked from commit 9508758)
happymaj11r pushed a commit to happymaj11r/openpilot that referenced this pull request May 4, 2026
@ajouatom
Revert "Implement Tire Trajectory feature and cleanup Path Modes (ajo…
5bdbd72 
…uatom#342)"

This reverts commit 9508758.

(cherry picked from commit 9aa4036)
ajouatom added a commit that referenced this pull request May 6, 2026
@ajouatom
tireTrajectory & dashboard error fix. (#351)
178b8ac 
* Reapply "Implement Tire Trajectory feature and cleanup Path Modes (#342)"

* late lat enable

* fix..

* fix..

* car dashboard error.

* fix dashboard.. error.

* fix lat enabled..
happymaj11r pushed a commit to happymaj11r/openpilot that referenced this pull request May 8, 2026
@ajouatom
tireTrajectory & dashboard error fix. (ajouatom#351)
9e62445 
* Reapply "Implement Tire Trajectory feature and cleanup Path Modes (ajouatom#342)"

* late lat enable

* fix..

* fix..

* car dashboard error.

* fix dashboard.. error.

* fix lat enabled..

(cherry picked from commit 178b8ac)
dongho74s pushed a commit to dongho74s/carrotpilot that referenced this pull request May 9, 2026
@janpoo6427
Update target and navigation icon conditions (ajouatom#347)
584e36d 
Refactor LCA icon value assignments for clarity

web (ajouatom#350)

* queue

* locale

tireTrajectory & dashboard error fix. (ajouatom#351)

* Reapply "Implement Tire Trajectory feature and cleanup Path Modes (ajouatom#342)"

* late lat enable

* fix..

* fix..

* car dashboard error.

* fix dashboard.. error.

* fix lat enabled..

boot logs (c3-wip) (ajouatom#355)

* scorn log

* F

* ff

* ff

* text cc

* font size

angle steer: adjust steering pressed return torque timer. 5 -> 50

web (ajouatom#356)

* default set

* button locate

* button

* ui f

* ui

* profile add

* ff

* f

* ff

* css

* f

* ui

* color

* f

* f

* f

* ff

* f

* f

* drop menu

* f

* ani

* a

* a

css (ajouatom#357)

* fx

* fix

* fix

Update carstate.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ChanHoya @ajouatom @chanhojung