Skip to content

docs: add measured M2 screw list and shopping list (sg90) - #41

Merged
jamro merged 1 commit into
jamro:mainfrom
mjspe:docs/screw-list-sg90
Sep 19, 2026
Merged

jamro merged 1 commit into
jamro:mainfrom
mjspe:docs/screw-list-sg90

Conversation

@mjspe

@mjspe mjspe commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

The docs said exact screw lengths/counts "aren't fully documented yet" in three places. This PR documents them.

What I did: ran a geometric census over every shipped parts/sg90/3mf/*.3mf mesh, detecting cylindrical features matching the M2 pilot spec (~Ø2.1 mm). The detector was validated against the labeled holes in ScrewSizingTest.3mf (recovered 8 of 9 labeled diameters exactly).

Findings added to 3d_models/README.md:

  • Per-part pilot table: 27 screw locations across 11 parts (depths 2.5–12.8 mm)
  • The 5 deep chest channels (14–23 mm) are servo-wire routing, not screws
  • Only the head ships servo-tab pilots; the other 4 servos are pocket-fit + horn-screw captured
  • No counterbores/countersinks anywhere → pan/button-head M2s
  • Shopping list: ~30× M2×6 + ~5× M2×12 (one M2 assortment kit covers it)
  • Servo horn screws come in the servo bags (5 total, one per joint)

Also updated the two stale "not documented yet" pointers in docs/getting-started.md and docs/hardware-for-software-engineers/06-3d-printing-and-mechanical-build.md to link the new screw list.

Counts were measured from exported meshes rather than the Fusion source, and the README says so — treat as a verified starting point, test-fit as you go.

Summary by CodeRabbit

  • Documentation
    • Added measured screw specifications and pilot-hole guidance for SG90-based builds.
    • Documented an approximate per-robot shopping list: 30 M2×6 and 5 M2×12 pan- or button-head screws.
    • Linked assembly and screw guidance across the 3D models, getting-started, and mechanical build documentation.
    • Clarified that a complete step-by-step assembly guide is not yet available.

Census of the shipped sg90 meshes (~2.1 mm pilots): 27 screw
locations across 11 parts. Shopping list: ~30x M2x6 + ~5x M2x12
pan/button head. Notes that the 5 deep chest channels are
servo-wire routing (not screws) and only the head ships
servo-tab pilots.
@coderabbitai

coderabbitai Bot commented Sep 18, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 72da4963-efb0-486c-951b-b94055b91052

📥 Commits

Reviewing files that changed from the base of the PR and between c45b281 and ae2c24c.

📒 Files selected for processing (3)
  • 3d_models/README.md
  • docs/getting-started.md
  • docs/hardware-for-software-engineers/06-3d-printing-and-mechanical-build.md

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

The documentation now defines measured SG90 screw specifications, pilot details, and approximate shopping quantities. The getting-started and mechanical-build guides link to this information.

Changes

Screw documentation

Layer / File(s) Summary
Screw reference and measurements
3d_models/README.md
Adds M2 screw specifications, SG90 pilot measurements, servo retention notes, and an approximate shopping list.
Build guide references
docs/getting-started.md, docs/hardware-for-software-engineers/06-3d-printing-and-mechanical-build.md
Updates both guides with the approximate M2×6 and M2×12 quantities and links to the README screw section.

Priority: ⚪ Not assessed

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Suggested reviewers: jamro

Merge Risk: ⚪ Minimal · up to ae2c2

The updated screw guidance and cross-document links are consistent, with no identified impact on builds or runtime behavior.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding measured M2 screw and shopping-list documentation for the SG90 build.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jamro

jamro commented Sep 19, 2026

Copy link
Copy Markdown
Owner

This is fantastic! thank you! I definitely didn’t expect someone to run a geometric census over all the exported models to fill this gap 😄

Really appreciate the thorough work here. This is exactly the kind of contribution that makes the project better for everyone building it. Merging!

@jamro
jamro merged commit 804529a into jamro:main Sep 19, 2026
3 checks passed
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

Successfully merging this pull request may close these issues.

2 participants