Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Shumin/landmark #5

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open

Shumin/landmark #5

wants to merge 26 commits into from

Conversation

Shumin326
Copy link
Collaborator

Finalized this semester's work on landmark detection. Main work:

  1. Use landmark detection and compare with expected landmark position to update particle weights.
  2. Vectorized the whole process so it now should be capable to run onboard.

Future work(VERY IMPORTANT)

After debugging, the seeable function is correct but the transformation from landmark in world coordinate to image pixel is not accurate, which yields the issue that either camera calibration is wrong(extrinsic matrix is wrong) or camera intrinsic matrix is wrong. Need to look into this otherwise landmark detection would not be benefitial.

NOTE: Landmark detection update is by default disabled. I will improve the module and enable it in the future.

@Shumin326 Shumin326 requested a review from shineyruan December 19, 2021 22:15
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