[Fancy Stack-up][Fixed] KiCad 9 inner layer computation

Fixes #870

Author: set-soft

CHANGELOG.md

@@ -33,7 +33,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Huge memory usage for panels (#842)
 - Panelize: vjustify wrong choices (were the same as hjustify) (#846)
 - THT Resistors: Problems when using the STEP version of the model (#856)
-- Fancy stackup: Limit the decimals of the copper weight to 2 (#864)
+- Fancy stackup:
+  - Limit the decimals of the copper weight to 2 (#864)
+  - Buried vias issues on KiCad 9 (#870)
+
 
 ### Changed
 - Filtered KiBot warnings are informed in the debug output (See #817)

docs/source/Changelog.rst

@@ -61,7 +61,10 @@ Fixed
 -  Panelize: vjustify wrong choices (were the same as hjustify) (#846)
 -  THT Resistors: Problems when using the STEP version of the model
    (#856)
--  Fancy stackup: Limit the decimals of the copper weight to 2 (#864)
+-  Fancy stackup:
+
+   -  Limit the decimals of the copper weight to 2 (#864)
+   -  Buried vias issues on KiCad 9 (#870)
 
 Changed
 ~~~~~~~

kibot/gs.py

@@ -1119,14 +1119,18 @@ def inner_layer_index(id):
 
     @staticmethod
     def copper_layer_to_ordinal(n):
-        ordinal = pcbnew.CopperLayerToOrdinal(n)
-        # Adjust to the current PCB
-        if ordinal == pcbnew.CopperLayerToOrdinal(pcbnew.B_Cu):
-            ordinal = GS.board.GetCopperLayerCount()-1
-        return ordinal
+        if GS.ki9:
+            ordinal = pcbnew.CopperLayerToOrdinal(n)
+            # Adjust to the current PCB
+            if ordinal == pcbnew.CopperLayerToOrdinal(pcbnew.B_Cu):
+                ordinal = GS.board.GetCopperLayerCount()-1
+            return ordinal
+        # <= 8
+        return GS.board.GetCopperLayerCount()-1 if n == pcbnew.B_Cu else n
 
     @staticmethod
     def ordinal_to_copper_layer(n):
+        # Only for KiCad 9+
         if n == GS.board.GetCopperLayerCount()-1:
             return pcbnew.B_Cu
         if n == 0:

kibot/misc.py

@@ -340,6 +340,7 @@
 W_DEFNOSTR = '(W172) '
 W_CONVPDF = '(W173) '
 W_MISSWRL = '(W174) '
+W_STACKUP = '(W175) '
 # Somehow arbitrary, the colors are real, but can be different
 PCB_MAT_COLORS = {'fr1': "937042", 'fr2': "949d70", 'fr3': "adacb4", 'fr4': "332B16", 'fr5': "6cc290"}
 PCB_FINISH_COLORS = {'hal': "8b898c", 'hasl': "8b898c", 'imag': "8b898c", 'enig': "cfb96e", 'enepig': "cfb96e",

kibot/pre_draw_fancy_stackup.py

@@ -13,7 +13,7 @@
 from .layer import Layer
 from .optionable import Optionable
 from .macros import macros, document, pre_class  # noqa: F401
-from .misc import VIATYPE_THROUGH, VIATYPE_BLIND_BURIED, VIATYPE_MICROVIA, W_NOVIAS
+from .misc import VIATYPE_THROUGH, VIATYPE_BLIND_BURIED, VIATYPE_MICROVIA, W_NOVIAS, W_STACKUP
 from . import log
 import pcbnew
 logger = log.get_logger()
@@ -146,6 +146,9 @@ def __init__(self):
         self.layer_type = ''
         self.gerber = ''
 
+    def __str__(self):
+        return f"{self.layer} mat: {self.material} thick: {self.thickness} use: {self.layer_type} type: {self.type}"
+
 
 def draw_core(g, x, y, w, h, layer, offset, border_w=10000):
     y = y-int(h*0.5)+offset
@@ -647,16 +650,16 @@ def create_stackup_matrix(stackup, via_layer_pairs, draw_vias):
 
 
 def get_layer_number(stackup, number):
-    copper_num = number
-    if number == pcbnew.B_Cu:
-        copper_num = GS.board.GetCopperLayerCount() - 1
+    copper_num = GS.copper_layer_to_ordinal(number)
 
     i = 0
     for n, layer in enumerate(stackup):
         if layer.type == 'copper':
             if i == copper_num:
                 return n
             i += 1
+    logger.warning(W_STACKUP+f"Wrong stack up? (layer id: {number} not found)")
+    return n
 
 
 def update_drawing(ops, parent):