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

Request for a new release of MoveIt into Humble #3012

Open
Danilrivero opened this issue Sep 25, 2024 · 22 comments
Open

Request for a new release of MoveIt into Humble #3012

Danilrivero opened this issue Sep 25, 2024 · 22 comments
Assignees
Labels
enhancement New feature or request

Comments

@Danilrivero
Copy link

Is your feature request related to a problem? Please describe.

Following up with the release of #2928 and #3002 to main (aka Jazzy) I have checked the Release Process for releases to older versions but could not resolve the cadence for these.

Describe the solution you'd like

The last Release for Humble dates back to a year ago with the release of 2.5.5. There are many fixes that could benefit those using this version right now, which is our case with our Stack.

Describe alternatives you've considered

If the only version being supported for new binaries is Jazzy and there are no plans for Humble it would push us to use the Source version until we migrate to this latest version.

Additional context

Thanks to everyone involved once again.

@TSNoble
Copy link
Contributor

TSNoble commented Nov 10, 2024

I'd be very interested in getting #3061 (backported), #3075 (backported) and #3077 (backported) as our usage of moveit is very pilz + subframe planning heavy, and it's much easier integrating the binaries into our stack.

@rr-tom-noble
Copy link
Contributor

@sea-bass @sjahr Do you know how to kickstart this process? It seems like it needs someone with write access to the repos?

@Danilrivero
Copy link
Author

ROS 2 releases that are still being supported should see some sort of cadence as well even though they would not have the frequency of the newest release, I don't know if this is defined somewhere.

The release process for Humble would definitely require some work for the changelogs, it's been a while since the last release

@sea-bass
Copy link
Contributor

The whole process is listed here:
https://docs.ros.org/en/humble/How-To-Guides/Releasing/Subsequent-Releases.html

Anyone can auto generate a changelog and release locally with this process, but the generation of a tag and then submitting the PR to rosdistro does require being part of the release organization -- which I and others can help with.

The other note is because Humble is not the main version, this should specifically be a patch release (2.5.6).

@rr-tom-noble
Copy link
Contributor

I'm happy to give this a go once the humble backports for the issues I mentioned are merged. Hopefully in the next couple of days. @sea-bass would it be worth putting out some sort of notification in case there are other soon-to-merge PRs people want to include?

@sea-bass
Copy link
Contributor

Thanks @rr-tom-noble -- I'll take care of the release once all these backports/fixes are in.

Worth asking since you're bringing a lot of great changes recently: do YOU have any more backports lined up before we cut one?

@sea-bass sea-bass self-assigned this Nov 13, 2024
@rr-tom-noble
Copy link
Contributor

@sea-bass Haha fair point! I don't have any immediate fixes planned. A colleague of mine is working on fixing an issue related to updating collision object colours, but they're quite busy and are happy to wait. I'm also quite busy at work, but will try to get the humble backport of #3077 passing tonight.

@sea-bass
Copy link
Contributor

sea-bass commented Nov 13, 2024

Oh, it wouldn't happen to be related to this fix I made last year, would it?

#2549

Would be nice if that just needs a backport.

@rr-tom-noble
Copy link
Contributor

Ah I'm not sure. @rr-mark @rr-aiden ?

@rr-mark
Copy link

rr-mark commented Nov 13, 2024

@sea-bass That looks like some of what we need, thanks.

We're also trying to recolour attached collision objects, and make collision objects (including attached collision objects) transparent/translucent.

Your PR certainly looks like the right bit of the code for us to look at.

@sjahr
Copy link
Contributor

sjahr commented Nov 14, 2024

@rr-tom-noble @rr-mark @Danilrivero Thanks for all your contributions here recently. They are much appreciated! It would be nice to get in touch and sync our collaboration on MoveIt more so we can help you to get your changes even faster into the code base. Would you be available to join the next public maintainer meeting? We're going to invite publicly on discourse but I wanted to notify you earlier so you might be able to block it in your calendar if you're interested

MoveIt Maintainer Meeting
Thursday, November 28 · 5:00 – 6:00pm
Time zone: Europe/Berlin
Google Meet joining info
Video call link: https://meet.google.com/fpo-srqg-feg
Or dial: ‪(US) +1 347-486-5750‬ PIN: ‪644 991 049‬#

@rr-tom-noble
Copy link
Contributor

@sjahr Thanks for the invite! I'll put it in my calendar 👍

@rr-mark
Copy link

rr-mark commented Nov 14, 2024

@sjahr Yes, that would be good.

@rr-tom-noble
Copy link
Contributor

rr-tom-noble commented Nov 15, 2024

@sea-bass Given #3077's humble backport is causing some issues, I'm happy to leave this out of the release while I investigate.

We have the ability to build individual moveit packages from source, but this has been causing issues with the current release (2.5.5) when building some packages from humble-based branches with custom fixes (e.g. #3083).

I think this is because the humble branch contains additional backports which make the binaries we produce from source incompatible with the 2.5.5 binaries (my mistake for basing our custom branches on humble rather than the 2.5.5 tag).

Generating a new release now should resolve that latter issue for us, and mean needing to port fewer unreleased fixes across to our custom branches.

Additionally, would it be possible to commit to a regular release cadence for humble as mentioned by @Danilrivero? I'll be working to fix the #3077 backport, and it seems there are still issues with subframes that haven't been resolved #3089, so I'll likely have a few more PRs in the works

@sea-bass
Copy link
Contributor

I don't think it's a good idea to mix binaries and source builds.

If you want to test out your updated branches, you should do something to the effect of sudo apt remove ros-humble-moveit*

The problem is because of how apt works, you may need to stick a few more repos like moveit_msgs and moveit_resources in your workspace to prevent rosdep from reinstalling MoveIt binary packages.

If things work that way, should be good. You can refer to the .repos file in the moveit2_tutorials repo for what you may need.


Also re: regular cadence for Humble releases, it's a good idea, but as you've seen the active maintainer team is a bit thin right now. I don't even use MoveIt currently, but I'm helping here and there. I'll at least cut the next release soon.

@rr-tom-noble
Copy link
Contributor

Understood and no worries about the release cadence. Apologies for pushing extra stuff into your workload recently. I greatly appreciate the support you've provided, and that you're helping to manage this release 🙂

@sea-bass
Copy link
Contributor

sea-bass commented Nov 17, 2024

The release is good to go, but when I do bloom-release I get a permissions error, despite being in the release team.

$ bloom-release --rosdistro humble moveit

ROS Distro index file associate with commit '5182bb14955c334a5785d5b591b99d19f2924c0a'
New ROS Distro index url: 'https://raw.githubusercontent.com/ros/rosdistro/5182bb14955c334a5785d5b591b99d19f2924c0a/index-v4.yaml'
==> Fetching 'moveit' repository from 'https://github.com/ros2-gbp/moveit2-release.git'
Cloning into '/tmp/tmpiccze4br'...
remote: Enumerating objects: 140563, done.
remote: Counting objects: 100% (4997/4997), done.
remote: Compressing objects: 100% (987/987), done.
remote: Total 140563 (delta 4454), reused 4010 (delta 4010), pack-reused 135566 (from 1)
Receiving objects: 100% (140563/140563), 46.36 MiB | 19.14 MiB/s, done.
Resolving deltas: 100% (42016/42016), done.
==> Setting release repository remote url to '[email protected]:ros2-gbp/moveit2-release.git'
==> git remote set-url origin [email protected]:ros2-gbp/moveit2-release.git
==> Testing for push permission on release repository
==> git remote -v
origin  [email protected]:ros2-gbp/moveit2-release.git (fetch)
origin  [email protected]:ros2-gbp/moveit2-release.git (push)
==> git push origin master --dry-run
[email protected]: Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
Cannot push to remote release repository.
Hint: If you just typed in your username/password and you have two-factor authentication,see:
  http://wiki.ros.org/bloom/Tutorials/GithubManualAuthorization

If anyone else with permissions wants to do this, please feel free.

@TSNoble
Copy link
Contributor

TSNoble commented Nov 18, 2024

Thanks for all of you work @sea-bass.

Given the maintainer team is thin, I'd be happy to try leading future releases (at least until we upgrade to jazzy, and up to the point where write access is needed) since we're using humble quite a lot now, and the binaries are so convenient for us.

I still have an outstanding bug to work on, and I'm feeling like refactoring parts of the pilz code in the future so it's easier to maintain, So I may be pushing for 2.5.7 soon(ish) 😛

@rr-tom-noble
Copy link
Contributor

Also @sea-bass do you know what ssh key you were using and (if the commands were run inside a docker container, were they mounted)?

@sea-bass
Copy link
Contributor

sea-bass commented Nov 18, 2024

Also @sea-bass do you know what ssh key you were using and (if the commands were run inside a docker container, were they mounted)?

I have a key generated for Bloom, used in my ~/.config/bloom, and running locally, so no mounting issues.

Moreover, I've run bloom-release on other packages such as generate_parameter_library and pick_ik before.

So not sure.

@sea-bass
Copy link
Contributor

Nah, you were right @rr-tom-noble -- I had SSH keys set up for my old work laptop, but never did it for my personal one 🤦🏻

PR to rosdistro is now queued up with the release!

ros/rosdistro#43534

@rr-tom-noble
Copy link
Contributor

Nah, you were right @rr-tom-noble -- I had SSH keys set up for my old work laptop, but never did it for my personal one 🤦🏻

Haha I only suggested as I've done the same many times 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

6 participants