Skip to content

looking at this fork... DO NOT MERGE #19

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

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

Conversation

oroulet
Copy link
Collaborator

@oroulet oroulet commented Jan 17, 2017

@martinbjerge any reason you have not done any pull requrest for some of these changes?

@oroulet oroulet changed the title looking at this fork... looking at this fork... DO NOT MERGE Jan 17, 2017
@martinbjerge
Copy link

No, we just stopped using urx, hence is slipped out of my focus, and I’m new to git and therefore not familiar with the processes.
Should I do something?

@oroulet
Copy link
Collaborator Author

oroulet commented Jan 17, 2017

no I will have a look. Just out of interest how do you control the robot?

@martinbjerge
Copy link

Not exactly sure what you mean, but today we use a python 3 framework that we developed our self.
https://bitbucket.org/RopeRobotics/ur-interface/

Hope this was the answer you where looking for :-)

@oroulet
Copy link
Collaborator Author

oroulet commented Jan 17, 2017

@martinbjerge yes it was. Why did you create your own? What is goal? Also just out of interest ...

@martinbjerge
Copy link

Manly do to license, urx is GPL and I have made mine MIT license.
The I also needed the RTDE and dashboard data connection.
Then I would like to have something that was more true to the script manual, hence if you know the script manual it would also be easy to start using this python library.

@kritzikratzi
Copy link

@martinbjerge is your library online somewhere?

@martinbjerge
Copy link

@martinbjerge is your library online somewhere?

Yes, here: https://github.com/martinbjerge/python-urx
and here: https://bitbucket.org/RopeRobotics/ur-interface/

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.

4 participants