-
Notifications
You must be signed in to change notification settings - Fork 541
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
Comments
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 |
The whole process is listed here: 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). |
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? |
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 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. |
Oh, it wouldn't happen to be related to this fix I made last year, would it? Would be nice if that just needs a backport. |
@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. |
@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 |
@sjahr Thanks for the invite! I'll put it in my calendar 👍 |
@sjahr Yes, that would be good. |
@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 |
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 The problem is because of how apt works, you may need to stick a few more repos like If things work that way, should be good. You can refer to the 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. |
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 🙂 |
The release is good to go, but when I do
If anyone else with permissions wants to do this, please feel free. |
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) 😛 |
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 Moreover, I've run So not sure. |
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 |
Haha I only suggested as I've done the same many times 😁 |
Shall we close this issue out now, and raise new issues as-and-when we need them? (I personally have my eye on #3118, but feels a bit extreme to push for another release so soon for just one commit 😅 ) @Danilrivero would you also be happy with this? I'm wondering if it'd also be possible to create some kind of pinned issue for Humble releases, which can be updated with links to the individual issues like this one (and future ones). It'd be good for tracking the status of humble, I think, and easier to find where open discussions about the latest release are (or if there is one at all) -- this one was already quite far down the list when I happened to find it |
Oh yes, I'll close this. Re: your suggestion, we may be able to use GitHub discussions for that. Let me make something. |
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.
The text was updated successfully, but these errors were encountered: