Skip to content

Conversation

@lukas-runge
Copy link
Contributor

Hi @jbtronics,

first of all - thank you for developing and maintaining this incredible piece of software. 🙏

When initially adding all my parts to Part-DB with the defaults provided, I just discovered that categories get created with excluded_from_sim set to true. When also creating all parts with the default of null (which means inherit from category) then all parts are per default excluded from simulation when adding them to KiCad via the API endpoint. This means that in older versions of KiCad (pre v9) newly placed parts are initially shown crossed out

Bildschirmfoto 2026-01-15 um 19 27 48

and in newer versions (v9 and newer) they get outlined and shown with an additional tag on the bottom right side

Bildschirmfoto 2026-01-15 um 19 29 26

indicating that they are excluded from simulation.

This is rather annoying as changing this now requires manual adjustment of that setting for every single category in existence. This could only be made easier for existing users by providing some kind of batch editing feature which as far as I know is not a thing.

To at least mitigate this inconvenience for new users and new categories that existing users create, this PR proposes to set excluded_from_sim to null for new categories and default to false in the helper function if the property is not set.

Best regards,
@lukas-runge

@codecov
Copy link

codecov bot commented Jan 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.21%. Comparing base (0a8199d) to head (26b6fa7).
⚠️ Report is 10 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1192      +/-   ##
============================================
- Coverage     58.22%   58.21%   -0.02%     
  Complexity     7428     7428              
============================================
  Files           582      582              
  Lines         23641    23641              
============================================
- Hits          13766    13763       -3     
- Misses         9875     9878       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jbtronics
Copy link
Member

I would not really call it a bug, but sounds reasonable to change the default behavior here.
Thanks.

@jbtronics jbtronics merged commit af81e15 into Part-DB:master Jan 18, 2026
16 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