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

How is this feedstock maintained? #420

Closed
peci1 opened this issue Oct 23, 2023 · 5 comments
Closed

How is this feedstock maintained? #420

peci1 opened this issue Oct 23, 2023 · 5 comments

Comments

@peci1
Copy link
Contributor

peci1 commented Oct 23, 2023

@Tobias-Fischer FYI - this change also commented out all the other packages since we were iterating on this and didn't need to keep building those. Probably you didn't mean to also merge those changes?

Originally posted by @defunctzombie in #345 (comment)

No, this was deliberate, all good :)

Thanks again for the contribution!

Originally posted by @Tobias-Fischer in #345 (comment)

@peci1
Copy link
Contributor Author

peci1 commented Oct 23, 2023

@Tobias-Fischer As a "maintainer" ("adder") of some of the commented out packages, I'm quite surprised. I added the packages because I wanted to have them in robostack. Now they're gone without anyone notifying me. This is super-bad for my further will to support robostack/prefix.dev or my will to create any of our systems that would depend on it. We were even thinking about using robostack for teaching, but for that, we need predictability and reliability.

@peci1 peci1 changed the title How is this feedstock maintained??? How is this feedstock maintained? Oct 23, 2023
@Tobias-Fischer
Copy link
Collaborator

Hi @peci1 - thanks for getting in touch!

Regarding the changes in the yaml files: The commented-out packages are "artefacts" on how the yaml files are being processed with vinca. We have a flag full_rebuild, if this is set to false, then only a select subset of packages are rebuilt. So commenting packages out does not mean that they are not supported/maintained anymore, just that they didn't need to be built at this point in time as they have already been built. You can check out our https://robostack.github.io/Contributing.html for more details.

@traversaro is currently pushing toward rebuilding all currently supported packages in #418. You can see currently supported packages at https://robostack.github.io/noetic.html. Which packages exactly are "gone"?

Please note that RoboStack's core contributors are less than a handful of people who have full-time jobs outside of RoboStack. We are always looking for more hands to help out, you would be more than welcome to contribute. It would be great to see RoboStack being used more for teaching - some unis are already using it in their units.

Best, Tobi

@peci1
Copy link
Contributor Author

peci1 commented Oct 24, 2023

Ah, right. Thanks for reminding me about this procedure. It's been some time since I last contributed so I totally forgot it works like this.

Maybe to decrease the chance other contributors are surprised, there could be some comments added to the vinca files saying that the commented-out packages in some parts are still supported?

What it the current procedure if a community-added package starts having build problems? Do you git-blame to find the author and contact him/her? Or do you try to fix the build yourself first? Can it happen that a package is removed from the distribution?

And how do you figure out a package needs to be rebuilt? Is that on request, plus a few full rebuilds yearly?

Which packages exactly are "gone"?

None from the anaconda hosting. I was just wondering whether their commenting out from the vinca files means they are no longer supported.

Please note that RoboStack's core contributors are less than a handful of people who have full-time jobs outside of RoboStack. We are always looking for more hands to help out, you would be more than welcome to contribute.

Hmm, I hoped that after the announcement of https://discourse.ros.org/t/new-way-of-installing-ros/33929 , there was some task force helping robostack a bit more...

It would be great to see RoboStack being used more for teaching - some unis are already using it in their units.

We'll definitely have a look at this possibility before the next semester as we're still struggling to find a simple yet working solution for students with all OSs.

@Tobias-Fischer
Copy link
Collaborator

Hi @peci1 - we welcome contributions to our documentation, this is something we should definitely improve!

We typically try to fix build issues ourselves first, if it is too tricky we will be reaching out to people who have requested/added the package in question. We try not to remove packages anymore that are built in robostack-staging; that said at some point we tried to build as many packages as possible, and that became too much of a maintenance overload.

Yes, packages are rebuild on request + sporadic full rebuilds.

I wish there was a bigger task force :)

@Tobias-Fischer
Copy link
Collaborator

#427 has been merged

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

No branches or pull requests

2 participants