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

Found the issue with this homography #61

Open
DavidDePauw1 opened this issue Mar 11, 2024 · 1 comment
Open

Found the issue with this homography #61

DavidDePauw1 opened this issue Mar 11, 2024 · 1 comment

Comments

@DavidDePauw1
Copy link

MatOfPoint2f cameraPoints = new MatOfPoint2f(

You have a few incorrect points labelled here. I can tell just from looking at the numbers here that the line with "94" as the pixel x coordinate is missing a digit (should be 940-somthing), and you are missing several negatives in the y meters component in the last few points. The x values all seem plausible. This would explain why the distances we were seeing were fairly close, but the left-right alignment was very off.

Please try to redo this as soon as you have an opportunity. I would recommend starting with just 4 labelled points near the corners of the image and testing to see if the homography comes out correctly. Once we know those 4 are correct, we can add a few more points to help reduce any noise that exists in the labelling process.

I'd really like to get auto-pickup working before Nick gets to the meeting tonight so we can start right on autos when he gets there. Once the homography is fixed, we will need to deal with the double-detection issue we were seeing yesterday, but then we should have everything we need for auto-pickup to work as intended.

@Alex-idk

@Alex-idk
Copy link
Contributor

I'll try to get those new points done after school today. I also just realized I'm applying a camera transform added to the homography that is already transformed.

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

No branches or pull requests

2 participants