INTI-CMNB / KiBot

KiCad automation utility
GNU Affero General Public License v3.0
571 stars 67 forks source link

[HOW] DRC exlusiones stoped working #653

Closed MarkusSBS closed 2 months ago

MarkusSBS commented 2 months ago

DRC stoped working for me and my jobs are failing.

Jobs for repositorys worked fine 2 monthe ago and without any changes fail if i rerun the job.

I have testet it with kicad 7 and i migrated a projekt to kicad 8 and have the same result.

is there a migration step i missed? I have no errors and some exlusiones in my local kicad DRC scan but all exlusions are reportet as errors.

MarkusSBS commented 2 months ago

here my kibot test projekt. I just rerun the pipeline: image

set-soft commented 2 months ago

DRC stoped working for me and my jobs are failing.

Jobs for repositorys worked fine 2 monthe ago and without any changes fail if i rerun the job.

Is hard to know the reason with no information about versions involved.

I have testet it with kicad 7 and i migrated a projekt to kicad 8 and have the same result.

is there a migration step i missed?

When migrating to KiCad 8 you should also migrate from run_drc to drc, a warning explains it. The drc preflight is cleaner and uses a fast mechanism.

I have no errors and some exlusiones in my local kicad DRC scan but all exlusions are reportet as errors.

DRC exclusions are stored in the project file (kicad_pro), they looks like this:

      "drc_exclusions": [
        "clearance|176385001|79614999|7e023245-2c2b-4e2b-bfb9-5d35176e88f2|666713b0-70f4-42df-8761-f65bc212d03b",
        "invalid_outline|165100000|80645000|e6d8b5bb-6f41-4222-9a89-b8a6280d0752|00000000-0000-0000-0000-000000000000",
        "silk_over_copper|153440000|84620000|3f5fe6b7-98fc-4d3e-9567-f9f7202d1455|f4f99e3d-7269-4f6a-a759-16ad2a258779"

They are verified in the regression tests, so I need an example where they fail to work.

MarkusSBS commented 2 months ago

drc_exclusions are included in the projekt file.

docker image ist image: name: ghcr.io/inti-cmnb/kicad7_auto_full:dev

I will send you a zip of the test repo

set-soft commented 2 months ago

Hallo @MarkusSBS !

This was related to a known KiCad bug that we had a workaround, but wasn't applied at all the needed places. In your particular case the set_text_variables triggered the issue.

Now should be fixed.