Skip to content

fusion_eagle: add THT/SMD Montage extra field - #592

Open
franz-ms-muc wants to merge 3 commits into
openscopeproject:masterfrom
Meisterschulen-am-Ostbahnhof-Munchen:feature/fusion-eagle-mount-type
Open

fusion_eagle: add THT/SMD Montage extra field#592
franz-ms-muc wants to merge 3 commits into
openscopeproject:masterfrom
Meisterschulen-am-Ostbahnhof-Munchen:feature/fusion-eagle-mount-type

Conversation

@franz-ms-muc

@franz-ms-muc franz-ms-muc commented Oct 13, 2026 •
edited
Loading

Copy link
Copy Markdown

Summary

  • Derive a THT/SMD "Montage" extra field for Eagle/Fusion boards from the footprint's <pad>/<smd> elements.
  • The vendor MOUNT attribute is only present on some library parts (e.g. Würth), so the BOM could not distinguish THT from SMD for all components.
  • Usable via --extra-fields "Montage".

Test plan

  • Generated the BOM for Leiterplatte_Trainer.brd with --extra-fields "Montage,PART-NUMBER" without errors.
  • Check the Montage column values in the browser.

🤖 Generated with Claude Code

Summary by Sourcery

Add reliable mounting-type classification for Fusion/Eagle BOM components based on their footprint elements.

New Features:

  • Add a configurable “Montage” extra field for Fusion/Eagle boards to classify components as THT, SMD, mixed, or mechanical.

Bug Fixes:

  • Improve component mounting-type identification when vendor-specific MOUNT attributes are unavailable.

Summary by CodeRabbit

  • New Features
    • Component details now indicate whether a package is through-hole, surface-mount, a mix of both, or mechanical based on its pads.

franz-ms-muc and others added 2 commits October 13, 2026 10:22
@franz-ms-muc @claude
fusion_eagle: add THT/SMD "Montage" extra field derived from footprin…
b164f4a 
…t pads

Eagle/Fusion parts only carry a MOUNT attribute when the library vendor
sets it. Derive THT/SMD from the package's <pad>/<smd> elements instead,
so it can be shown/grouped via --extra-fields "Montage".

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@franz-ms-muc @claude
fusion_eagle: mark packages without pads as MECH
5f655fe 
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 13, 2026 08:26
Copilot AI reviewed Oct 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@franz-ms-muc @claude
fusion_eagle: respect --normalize-field-case for Montage field
0bbef4c 
The BOM looks up extra fields by their lowercased name when case
normalization is enabled, so store the derived value under that key.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@qu1ck

qu1ck commented Oct 13, 2026

Copy link
Copy Markdown
Member

Not a fan of hardcoding an extra field that will be present on every board. Is it hard to do this on eagle/fusion side via some script and create this field when necessary?

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@franz-ms-muc @qu1ck