BOM generation broken on some PCBs about jlc-plugin-for-kicad HOT 9 CLOSED

Could you please test again with the latest release.

from jlc-plugin-for-kicad.

I just installed the latest release and now BOM and position generation is broken when "exclude DNP.." is selected.

from jlc-plugin-for-kicad.

Can confirm the same on my machine and setup. No BOM or position files are being generated.
Win11
Kicad 8.1
Fabrication-Toolkit 4.30

from jlc-plugin-for-kicad.

Does some part get excluded or the entire files do not get generated?
Could you please test again with the 4.4.0 pre-release.

from jlc-plugin-for-kicad.

Unfortunately, I only get the following outputs on the 4.4 pre-release:

• gerber.zip
• netlist.ipc
• designators.csv

Just to check and be sure, I went back to 4.0 and there I still get the BOM output.

from jlc-plugin-for-kicad.

I traced the bug to here: #128
The added condition seems to misbehave, some more exploration needed.

from jlc-plugin-for-kicad.

@bennymeg
I traced the issue to the commit 88ae87b

            skip_footprint = ((footprint.GetAttributes() & pcbnew.FP_EXCLUDE_FROM_POS_FILES)
                                or footprint.GetPadCount() == 0
                                or exclude_dnp
                                    and (footprint_has_field(footprint, 'dnp')
                                        or (footprint.GetValue().upper() == 'DNP')
                                        or getattr(footprint, 'IsDNP', bool))
            )

The issues I see:

1. getattr(footprint, 'IsDNP', bool) is not called anymore. It was getattr(footprint, 'IsDNP', bool)() but became getattr(footprint, 'IsDNP', bool) and since it is a function reference and is not None - it evaluates to True
2. not enough parentheses

How to fix:

            skip_footprint = ((footprint.GetAttributes() & pcbnew.FP_EXCLUDE_FROM_POS_FILES)
                                or footprint.GetPadCount() == 0
                                or (exclude_dnp
                                    and ((footprint_has_field(footprint, 'dnp')
                                        or (footprint.GetValue().upper() == 'DNP')
                                        or getattr(footprint, 'IsDNP', bool)())))
            )

I've tested the fix on KiCAD v8.01 with the plugin 4.3.0 + fixes.

In the master branch the fix should look like below:
https://github.com/bennymeg/Fabrication-Toolkit/blob/master/plugins/process.py#L162

            is_dnp = (footprint_has_field(footprint, 'dnp') 
                      or (footprint.GetValue().upper() == 'DNP') 
                      or getattr(footprint, 'IsDNP', bool)())

I.e. add parentheses to call the function returned by getattr()

from jlc-plugin-for-kicad.

Thank you @msalau, I'll fix it.

from jlc-plugin-for-kicad.

Fixed in v4.4.1

from jlc-plugin-for-kicad.