-
Notifications
You must be signed in to change notification settings - Fork 6
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
Feature/robot #188
Feature/robot #188
Conversation
The error you saw comes from L109 and L121 where the |
@@ -287,6 +287,7 @@ def init(self): | |||
self.make_buffers() | |||
self._update_matrix() | |||
self.instance_color = Color.from_rgb255(*next(self.viewer.scene._instance_colors_generator)) | |||
# TODO: check robot add instance color error | |||
self.scene.instance_colors[self.instance_color.rgb255] = self |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But what I can do here is to change to self.viewer.scene
just like L289 so it will be able to access it no matter when, I will push it to main now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thats very cool. Then should we just use self.viewer.scene
instead of self.scene
in other place as well? cus it access the same instance anyway?
@Licini and @tomvanmele this PR also ready! |
Yay. Thanks a lot for this PR. |
do you know, in this class,robot_object: RobotModelObject = viewer.scene.add(model,..... How to assemble a robot tool,and show in viewer.(not show in ros or rhino.) |
Thank you very much. I would like to add a tool for the robotic arm in this example and display it in the Viewer, instead of displaying the robotic arm on interfaces like ROS Rhino. We are using compas_view, But the tool I added in this way doesn't work. Can you design a robot that can attach a display example of the end tool as a reference. What issues do I need to pay attention to. from compas.datastructures import Mesh def make_rotate_function(index): for i, joint in enumerate(robot_object.configuration.joint_names): |
hi, could you post this on the forum, or on the issue tracker, instead of on a closed pull request? thanks! |
OK , I have submitted the issue,can you help handle it?thankyou
…---Original---
From: "Tom Van ***@***.***>
Date: Fri, Nov 8, 2024 19:24 PM
To: ***@***.***>;
Cc: ***@***.******@***.***>;
Subject: Re: [compas-dev/compas_viewer] Feature/robot (PR #188)
hi,
could you post this on the forum, or on the issue tracker, instead of on a closed pull request?
thanks!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: ***@***.***>
|
What type of change is this?