Skip to content

Comments

Calibration pipeline#216

Closed
michalj1 wants to merge 579 commits intomainfrom
s2025_EstCal_Calibration_PR
Closed

Calibration pipeline#216
michalj1 wants to merge 579 commits intomainfrom
s2025_EstCal_Calibration_PR

Conversation

@michalj1
Copy link
Collaborator

Includes the calibration pipeline, calibrated corner and front cameras, and now with extra scripts for creating files formatted like cameras.yaml for perception.

nvikramraj and others added 30 commits February 24, 2025 19:06
Ren990420 and others added 25 commits May 12, 2025 13:46
removed obstacle state
This is the default cone_detection.yaml. Verticals should modify this file and integrate with their planning & other functions.
…ed script to geotag images for individual files and gnss.txt file.
…ed script to geotag images for individual files and gnss.txt file.
…ation

S2025 perception cone detector 1st PR
…n_merge

Parking + Perception Integration. cleaned. ready to merge
Consolidating changes for PR to s2025 branch
Gazebo Entity detection - Agent and Obstacle Detection and Tracking - Feature Request
@michalj1 michalj1 requested a review from krishauser as a code owner May 14, 2025 06:22
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
25 Security Hotspots
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Copy link
Owner

@krishauser krishauser left a comment

Choose a reason for hiding this comment

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

PR is too large. Includes parking and summoning group's commits.

Copy link
Owner

Choose a reason for hiding this comment

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

Do not name this "new_cameras". Either it overwrites the old cameras or it doesn't belong.

Copy link
Owner

Choose a reason for hiding this comment

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

Your PR should not include anything from other groups.

Copy link
Owner

Choose a reason for hiding this comment

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

Do not add files from other groups.

Copy link
Owner

Choose a reason for hiding this comment

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

Remove

Copy link
Owner

Choose a reason for hiding this comment

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

Do not put test files in the GEMstack/onboard folder, move to testing

Copy link
Owner

Choose a reason for hiding this comment

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

Take this out

Copy link
Owner

Choose a reason for hiding this comment

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

Take this out

Copy link
Owner

Choose a reason for hiding this comment

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

Take this out

Copy link
Owner

Choose a reason for hiding this comment

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

Take this out

Copy link
Owner

Choose a reason for hiding this comment

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

Take this out

@michalj1 michalj1 closed this May 15, 2025
@michalj1
Copy link
Collaborator Author

Merging with the wrong branch, my bad

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.