Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fit_field not working on Win7 #144

Open
aiotech-pub opened this issue Oct 23, 2020 · 2 comments
Open

fit_field not working on Win7 #144

aiotech-pub opened this issue Oct 23, 2020 · 2 comments

Comments

@aiotech-pub
Copy link

I have a bom.ni local file where:

; Field name used to determine if a particular part is to be fitted
fit_field = FIT

then to test KiBoM I edited a capacitor (C4) on the schematic adding the FIT field filled with:
"+a,-b"

this is my command as typed in Kicad:
python "C:\Dati\Kicad\scripts\KiBoM-master/KiBOM_CLI.py" "-r b" "%I" "%O.html"

on the html file I would expect to don't see C4 listed, instead it is listed and the field on the FIT column contains "+a,-b"

this is my command as typed in Kicad:
python "C:\Dati\Kicad\scripts\KiBoM-master/KiBOM_CLI.py" "-r a" "%I" "%O.html"

on the html file I would expect to see C4 listed, it is listed and the field on the FIT column contains "+a,-b"

In general terms I would expect a different behaviour:
if -r present on the command row: in FIT only fields containing - and + will be considered and the output will be produced with or without them only reporting texts not containing - and + (for instance dnf), if all the fileds will be empty or no dnf (And similar) texts are not found the column will not br printed
if -r not present on the command row: only reporting texts not containing - and + (for instance dnf) if all the fileds will be empty or no dnf (And similar) texts are not found the column will not br printed

@eeintech
Copy link
Contributor

@aiotech-pub Have you tried this command instead:
python "C:\Dati\Kicad\scripts\KiBoM-master/KiBOM_CLI.py" -r a "%I" "%O.html"

It looks like there are two proposed fixes for the variant logic: #136 and #137
Could be related.

@set-soft
Copy link
Contributor

I'll suggest trying https://github.com/INTI-CMNB/KiBoM this fork has #137 fix

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

No branches or pull requests

3 participants