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

Display weed locations as white dots in camera augmentation on real robot #122

Closed
wants to merge 3 commits into from

Conversation

rodja
Copy link
Member

@rodja rodja commented Jul 17, 2024

This introduces another place for the work around we use already in other parts of the code until zauberzeug/rosys#13 is implemented. It allows the weeds of the plant provider to be rendered as white dots in the CameraCard.

@rodja rodja added the enhancement New feature or request label Jul 17, 2024
@rodja rodja added this to the 0.2.0 milestone Jul 17, 2024
@rodja rodja requested a review from pascalzauberzeug July 17, 2024 03:45
Copy link
Contributor

@pascalzauberzeug pascalzauberzeug left a comment

Choose a reason for hiding this comment

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

@rodja I changed the indentation in Commit 3351f17.
Is that correct?
And .projection() throws an error, because relative is a point and not a point3d

field_friend/interface/components/camera_card.py Outdated Show resolved Hide resolved
@rodja
Copy link
Member Author

rodja commented Jul 17, 2024

Yes, your indentation is correct. And I hopefully fixed the projection error... please try again @pascalzauberzeug.

@pascalzauberzeug
Copy link
Contributor

pascalzauberzeug commented Jul 17, 2024

Seems like it doesnt use correct coordinates, the screen coordinates seem too small to me and the weeds are not shown in the image.

Here is an example:
plant.position: Point(-0.036, 0.142)
position_3d: Point3d(-0.036, 0.142, 0.000)
relative: Point(-0.068, 0.015)
screen: Point(0.293, 0.015)

@rodja
Copy link
Member Author

rodja commented Jul 17, 2024

Ok. Let's not spend more time on this. The work on zauberzeug/rosys#130 will begin shortly.

@rodja rodja closed this Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants