universal_robot icon indicating copy to clipboard operation
universal_robot copied to clipboard

this package is outdated with calibration_devel branch

Open lianghongzhuo opened this issue 3 years ago • 6 comments

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?

lianghongzhuo avatar May 25 '21 13:05 lianghongzhuo

It is not very much out-of-sync. With https://github.com/fmauch/universal_robot/pull/10 the only effective differences are #562 and #538. I decided to do https://github.com/fmauch/universal_robot/pull/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...

fmauch avatar May 25 '21 14:05 fmauch

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 avatar May 25 '21 16:05 lianghongzhuo

Please at least claim this repo as unmaintained since it's outdated and people getting errors when jumping into this repo (https://github.com/ros-industrial/universal_robot/issues/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.

lianghongzhuo avatar Aug 06 '21 15:08 lianghongzhuo

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

fmauch avatar Sep 29 '21 10:09 fmauch

Thanks a lot for your efforts to fix this!

lianghongzhuo avatar Sep 29 '21 10:09 lianghongzhuo

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

ipa-nhg avatar Sep 29 '21 11:09 ipa-nhg

This issue can be closed by now.

fmauch avatar Oct 14 '22 14:10 fmauch