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

this package is outdated with calibration_devel branch #573

Closed
lianghongzhuo opened this issue May 25, 2021 · 7 comments · Fixed by #538
Closed

this package is outdated with calibration_devel branch #573

lianghongzhuo opened this issue May 25, 2021 · 7 comments · Fixed by #538

Comments

@lianghongzhuo
Copy link

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?

@fmauch
Copy link
Contributor

fmauch commented May 25, 2021

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 git pull to get the most recent state. In parallel we worked on getting the changes into this repository, what we already did to a great extent (see #371 for example).

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...

@lianghongzhuo
Copy link
Author

I know you did a good job on maintaining the ur driver and ur descriptions, thank you for that.
But why ros-indestrial maintainers did not react?

@lianghongzhuo
Copy link
Author

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.

@ipa-nhg ipa-nhg added the wrid21 label Sep 23, 2021
@fmauch
Copy link
Contributor

fmauch commented Sep 29, 2021

@ipa-nhg This could be closed as soon as #538 gets merged, I guess.

@lianghongzhuo
Copy link
Author

Thanks a lot for your efforts to fix this!

@ipa-nhg ipa-nhg linked a pull request Sep 29, 2021 that will close this issue
@ipa-nhg
Copy link
Member

ipa-nhg commented Sep 29, 2021

@fmauch yes, I think so. I linked the issue to get this one automatically closed once is merged

@fmauch
Copy link
Contributor

fmauch commented Oct 14, 2022

This issue can be closed by now.

@fmauch fmauch closed this as completed Oct 14, 2022
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 a pull request may close this issue.

3 participants