Geometry Check on BOP

[easyw]

Hi @Shackmeister and all developers,
I've just updated the 'cq_cad_tools.py' adding two new functions:

1. checkUnion(doc)
2. checkBOP(shape)

and a source file to run these functions (I included all the functions in the file so to be able to run it easily)
checkBOP-example.py
I attached here also a wrong STEP model, not correctly union-ed and with wrong geometries to make some tests.
resistor-wrong-BOP.zip
uploaded also on the repo...

It is suggested to generate/export a STEP model, then close the file generator and import the STEP file;
then run the checkUnion function to see if the STEP file is correcty union-ed in a single part and run the checkBOP function to see if there are errors on the STEP shape.

This is going to be a quite robust check on the generated files... still in some case I found BOP check is not perfect, but it is for sure much better than the previous state.

Requirements:

• FreeCAD 0.17 12703 or later

[Shackmeister]

this sounds cool :D can you please try and add this in the new Resistor repo? :) would be nice with an example

[easyw]

@Shackmeister
now the 'Resistor_SMD' repo has a Union & BOP check
NB the BOP check is working ONLY with FC0.17 12703 or later (it will be skipped for previous FC releases)

[poeschlr]

This means my freecad is too old. (For some reason the last build for fedora was 48 days ago.)
Well i can use my ubuntu vm. Maybe there is a newer version for this operating system.

[easyw]

@poeschlr
for BOP check this commit should be enough
FreeCAD/FreeCAD@2a6a58e
but we had some issue with CQ fillet in previous FC0.17 releases...
may be your release could work fine...
you can simple test BOP check with the example files I linked above

[easyw]

a working models generator with Union and BOP check is ready here:
https://github.com/easyw/kicad-3d-models-in-freecad/tree/master/cadquery/FCAD_script_generator/Resistor_SMD

[poeschlr]

I tested it with your example file and my freecad version indeed discovered that the resistor is invalid.
So i included this test in my updated phoenix contact scripts.