Add Space Robotics#3981
Conversation
Signed-off-by: Andrej Orsula <orsula.andrej@gmail.com>
|
unicorn |
There was a problem hiding this comment.
Pull request overview
Adds a new Awesome list entry for “Space Robotics” to the main Awesome directory under the Hardware category.
Changes:
- Added a nested “Space Robotics” entry under the existing “Robotics” item in the Hardware section.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Andrej Orsula <orsula.andrej@gmail.com>
|
Please note that $ npx awesome-lint https://github.com/AndrejOrsula/awesome-space-robotics
✖ Linting
README.md:50:3
✖ 50:3 Duplicate link: https://jplopensourcerover.com remark-lint:double-link
...
✖ 303:3 Duplicate link: https://jplopensourcerover.com remark-lint:double-linkHowever, this is purposeful. For instance, https://jplopensourcerover.com is referenced two times, as:
I believe this makes sense, as people searching for interactive demonstrations would otherwise not come across this website when browsing the awesome list. Please let me know if you think otherwise. Thank you. |
wolffcatskyy
left a comment
There was a problem hiding this comment.
One issue worth noting:
-
Entry placement: The guidelines state "Your entry should be added at the bottom of the appropriate category." The PR places this as a nested bullet under
Roboticsrather than at the bottom of theHardwaresection. The PR acknowledges this and offers to move it — I'd suggest following the guideline and placing it at the bottom of theHardwarecategory as a top-level entry, sincesindresorhus/awesomedoesn't use nested category structure for sub-topics. -
Description: The entry description says "Resources, hardware platforms, software frameworks, and algorithms tailored for robotic systems in space." — this reads more like a description of the list itself rather than the topic. Per the guidelines, it should describe the project/theme, not the list. Something like "Robotics systems designed to operate in outer space environments." would be more appropriate.
-
Logo check: The list appears to have a logo — looks good.
Overall this is a well-prepared PR with good checklist compliance and substantive reviews of other PRs.
🤖 This review was generated by Claude AI assisting a maintainer.
|
Really solid list! The depth of the taxonomy is impressive, especially the split between deployed, experimental, and open-source systems. The inclusion of terrestrial analogue sites and laboratory facilities is a nice touch that I haven't seen in other robotics lists. Multi-format output (HTML/PDF/Markdown) is a great addition too. A few observations:
Overall, this is well-researched and clearly maintained. |
|
Impressive and thorough list! A few compliance issues to address:
|
Signed-off-by: Andrej Orsula <orsula.andrej@gmail.com>
Signed-off-by: Andrej Orsula <orsula.andrej@gmail.com>
|
Thanks @wolffcatskyy, @be-next, and @luka2chat for the detailed reviews and helpful suggestions! I really appreciate you taking the time to help me improve the list. I've gone ahead and updated the PR to address the feedback. I moved the Space Robotics entry to the very bottom of the Hardware category as a top-level item, and I updated the description to "Robotic systems designed for space environments" so it properly describes the theme rather than the list itself. Over at awesome-space-robotics, I've also resolved the formatting and compliance issues you pointed out. I cleaned up the main heading to use standard Markdown, and dropped the redundant contribution text from the tagline. I also took the advice to flatten the taxonomy and removed the deepest levels of nesting, such that the document doesn't go deeper than four levels now. According to the guidelines (see below), the capitalized
Just a quick heads up that the awesome-lint CI still complains about the duplicate links, but that's intentional as mentioned above: #3981 (comment) Thanks again for all the guidance! Please let me know if anything else needs to be corrected. |
|
|
niche, well organized, the mission/algorithm split feels like it was built by someone in the field one thing I didnt see flagged yet: the HTML/PDF/MD icon row right below the description. on the mdbook site it makes sense, on github.com it sits where the TOC should follow and competes visually with the Awesome badge. bottom of the readme would be cleaner |
https://github.com/AndrejOrsula/awesome-space-robotics
This list curates resources, hardware platforms, software frameworks, and algorithms specifically tailored for robotic systems in space.
Reviewed PRs:
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
Requirements for your pull request
Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration. Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions. Comments pointing out lint violation are allowed, but does not count as a review.
Add Name of List. It should not contain the wordAwesome.Add SwiftAdd Software ArchitectureUpdate readme.mdAdd Awesome SwiftAdd swiftadd SwiftAdding SwiftAdded Swift- [iOS](…) - Mobile operating system for Apple phones and tablets.- [Framer](…) - Prototyping interactive UI designs.- [iOS](…) - Resources and tools for iOS development.- [Framer](…)- [Framer](…) - prototyping interactive UI designs#readme.- [Software Architecture](https://github.com/simskij/awesome-software-architecture#readme) - The discipline of designing and building software.Requirements for your Awesome list
That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
awesome-linton your list and fix the reported issues. If there are false-positives or things that cannot/shouldn't be fixed, please report it.main, notmaster.Mobile operating system for Apple phones and tablets.Prototyping interactive UI designs.Resources and tools for iOS development.Awesome Framer packages and tools.If you have not put in considerable effort into your list, your pull request will be immediately closed.
awesome-name-of-list.awesome-swiftawesome-web-typographyawesome-SwiftAwesomeWebTypography# Awesome Name of List.# Awesome Swift# Awesome Web Typography# awesome-swift# AwesomeSwiftawesome-list&awesomeas GitHub topics. I encourage you to add more relevant topics.Awesome Xand a logo withAwesome X. You can put the header image in a#(Markdown header) or<h1>.Contents, notTable of Contents.ContributingorFootnotessections.https://github.com/<user>/<repo>/community/license/new?branch=main&template=cc0-1.0(replace<user>and<repo>accordingly).licenseorLICENSEin the repo root with the license text.Licencesection to the readme. GitHub already shows the license name and link to the full text at the top of the repo.unicorn.contributing.md. The casing is up to you.Contributing, positioned at the top or bottom of the main content.Footnotessection at the bottom of the readme. The section should not be present in the Table of Contents.Example:
- [AVA](…) - JavaScript test runner.Node.js, notNodeJSornode.js.You can still use a CI for linting, but the badge has no value in the readme.
Inspired by awesome-fooorInspired by the Awesome projectkinda link at the top of the readme. The Awesome badge is enough.Go to the top and read it again.