-
Notifications
You must be signed in to change notification settings - Fork 1k
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
this package is outdated with calibration_devel branch #573
Comments
It is not very much out-of-sync. With fmauch#10 the only effective differences are #562 and #538. I decided to do fmauch#10 in order to make my fork more exchangeable with the upstream repo in case someone doesn't want to use MoveIt!. The history looks different, as my calibration_devel branch took a different path in order to make users able to always do a For example, the driver's CI pipeline already uses the version from this repo. For everything else, the mentioned PRs are in place. In open source software development things sometimes take a bit more time, as there are always people necessary to critically review any incoming suggestions and changes which is often done aside any day jobs. While I understand that the current situation between the repositories is not the easiest to understand and I also wish that things get merged quite soon, I think we are in a very good situation already. I think it is clearly documented that my fork is currently to be used for the driver. As soon as we can swap back to the upstream version, the driver's documentation will be updated and my fork will not have to be used anymore... |
I know you did a good job on maintaining the ur driver and ur descriptions, thank you for that. |
Please at least claim this repo as unmaintained since it's outdated and people getting errors when jumping into this repo (#494) while there exist a bug-fixed version (https://github.com/fmauch/universal_robot/tree/calibration_devel) of this repo but not getting merged. I don't know why. |
Thanks a lot for your efforts to fix this! |
@fmauch yes, I think so. I linked the issue to get this one automatically closed once is merged |
This issue can be closed by now. |
Hi, as you all know, this package is outdated with 100+ commit with https://github.com/fmauch/universal_robot/tree/calibration_devel
This is confusing and breaks a lot of things. Why not just synchronize all commits from Felix?
The text was updated successfully, but these errors were encountered: