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

Transition robot models to main repository #46

Open
Jmeyer1292 opened this issue Apr 25, 2017 · 8 comments
Open

Transition robot models to main repository #46

Jmeyer1292 opened this issue Apr 25, 2017 · 8 comments

Comments

@Jmeyer1292
Copy link
Contributor

The irb120, irb120t, and irb4400 have all been in this repository for > 2 years and should be transitioned to the main abb repo.

@AustinDeric
Copy link
Contributor

I would like to see all the *_experimental repos be merged with the main repo. They should still be clearly identified This would allow us to:

  • save time
  • focus efforts on 1 repo
  • allow the experimental robots to be tested because the main repos get more use.

@gavanderhoorn
Copy link
Member

Just a data point on the 'visibility' of (support) pkgs in experimental repositories: the pkgs in fanuc_experimental get about as much use as those in fanuc (judging by the nr of questions / support requests I receive about them), which I believe is mainly due to the fact that the ROS buildfarm indexes both repositories and both collections have ROS wiki pages (wiki/fanuc, wiki/fanuc_experimental) that also cross-reference each other.

I've recently submitted the abb_experimental repository for indexing by the buildfarm and created an initial wiki page for it as well (wiki/abb_experimental). Updating the abb_experimental metapackage to exec_depend on the appropriate pkgs should bring visibility of those on-par with the main repository.

@gavanderhoorn
Copy link
Member

@Jmeyer1292 wrote:

The irb120, irb120t, and irb4400 have all been in this repository for > 2 years and should be transitioned to the main abb repo.

Should they first be updated to follow the ABB naming conventions?

@Levi-Armstrong
Copy link
Member

Should they first be updated to follow the ABB naming conventions?

I agree with updating them to the latest naming convention prior to transitioning them to the main repository.

@Jmeyer1292
Copy link
Contributor Author

Well I look forward to reviewing the pull requests from the maintainers of this package!

@gavanderhoorn
Copy link
Member

I think the visibility concerns for the packages in this repository have been addressed now that the ROS doc indexer has rebuilt everything today. See wiki/abb_experimental.

@AustinDeric
Copy link
Contributor

Back to @Jmeyer1292's original comment, do we have requirements that need to be satisfied to move these robots to the main repo? Are there any measurable metrics?

@gavanderhoorn
Copy link
Member

Are there any measurable metrics?

No, not at this time.

do we have requirements that need to be satisfied to move these robots to the main repo?

yes.

The point of having things in an experimental repository is that it allows the maintainers to make any potentially breaking changes without prior notice, that it allows for some soak time for new packages and to make them available to users to expose them to use-cases that are not necessarily those of the original author / maintainer.

After a certain amount of time packages stabilise and could be moved to the main repositories. At that point, the regular rules start to apply: semantic versioning, backwards-compatibility guarantees (tick-tock, etc) and all the rest.

Tbh I don't quite get what the problem is with this scheme. If it's lower visiblity then I believe we've addressed that with the creation of wiki/abb_experimental. If you take a look at the repository stats you'll see that that resulted in quite some traffic. My experience with fanuc_experimental is similar.

If it's the fact that experimental repositories are not released and have to be build from sources: I would say that anyone doing anything serious with ROS will have a workspace and runs Catkin once in a while anyway. With a proper .rosinstall and / or documentation it's peanuts to build the packages. But if we really want to we could create releases for the experimental repository as well.

The fact that #45, #53, #54, #56, #60, #63, #64, #65 and #67 were submitted -- and just for the few support pkgs that were contributed in the past few months -- I think shows that letting things sit here for some time is not such a bad thing.

Perhaps we could quantify what "a certain amount of time" is. That could take away some of the apparent 'arbitrary-ness' of when things do get migrated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants