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

Prakarsh gsoc2022 front-end #183

Closed
wants to merge 12 commits into from
Closed

Prakarsh gsoc2022 front-end #183

wants to merge 12 commits into from

Conversation

pariaspe
Copy link
Collaborator

From #180
Not merge yet, work to be done (see #180 for further info)

@pariaspe
Copy link
Collaborator Author

pariaspe commented Mar 17, 2023

To do before merging:

  • Undo car beacon on visual landing exercise
  • Change QRs with aruco markers in the packages (package delivery)
  • Use Aruco instead of apriltag on turtlebot. Or get back to green plate.

@pawanw17
Copy link

pawanw17 commented May 21, 2023

Hi Pedro, I tested the package delivery exercise with Qr code from this commit bbb3db9

the QR code when translated using this

        detector = cv2.QRCodeDetector()
        GUI.showImage(img_frontal)
        GUI.showLeftImage(img_ventral)
        
        data, bbox, _ = detector.detectAndDecode(img_ventral)
        
        if data:
            a=data
            print(a)

translates to "(20,20)" This data can be used for coordinates more appropriately than Aruco makers which only give the Marker ID. I tested this in the exercise and it was detected and decoded just fine (see console).

image

Here's the updated solution for the exercise (sometimes it is not able to find the delivery place even though the coordinates are correct:
My_solution_updated.txt

EDIT: On repeated testing, I find that sometimes it takes quite a while to detect a QR Code.

@pariaspe
Copy link
Collaborator Author

pariaspe commented Jun 4, 2023

Closing, replaced with #190

@pariaspe pariaspe closed this Jun 4, 2023
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.

3 participants