Bugfix: PCB Records could not be visualized.

svg_utils.py added - Function to generate an arc path get_arc_path is implemented here
SchArc and SchEllipticalArc refer to get_arc_path from svg_utils.py
PCBArc.py added

Author: ChrisHoyer

docs/source/classes/Components.rst

@@ -10,6 +10,7 @@ Generic Component (Base)
     .. autoattribute:: LibFile
     .. autoattribute:: Name
     .. autoattribute:: Description
+    .. autoattribute:: Records
 
     .. automethod:: pyaltiumlib.libcomponent.LibComponent.read_meta
     .. automethod:: pyaltiumlib.libcomponent.LibComponent.draw_svg
@@ -23,4 +24,5 @@ Schematic Symbol
 PCB Footprint
 ==================
 
-.. autoclass:: pyaltiumlib.pcblib.footprint.PcbLibFootprint
+.. autoclass:: pyaltiumlib.pcblib.footprint.PcbLibFootprint
+

docs/source/classes/LibFile.rst

@@ -34,3 +34,4 @@ PCB Library File (PCBLib)
 
 .. autoclass:: pyaltiumlib.pcblib.lib.PcbLib
 
+    .. autoattribute:: Layers

docs/source/usage/release.rst

@@ -14,7 +14,7 @@ The code is available in `this repository <https://github.com/ChrisHoyer/pyAltiu
 File Format Coverage
 =======================
 
-Currently, the format is not completely supported, some aspects are not fully implemented.
+Currently, the format is not completely supported, some aspects are not fully implemented. Not all containers or records are known yet, so this list and the associated documentation may be incomplete.
 
 The coverage of containers of the Altium library file format is shown below:
 
@@ -197,7 +197,7 @@ The coverage of records/primitives of the Altium library file format is shown be
      - Coverage
    * - PCB Primitives
      - Arc
-     - Not Implemented
+     - Key features implemented
    * - PCB Primitives
      - Pad
      - Key features implemented

src/pyaltiumlib/__init__.py

@@ -7,7 +7,7 @@
 AUTHOR_EMAIL = '[email protected]'
 CYEAR = '2024-2025'
 
-__version__ = "0.1"
+__version__ = "0.2"
 __author__ = "Chris Hoyer <[email protected]>"
 
 import os

src/pyaltiumlib/base.py

@@ -53,7 +53,9 @@ def __init__(self, filepath: str):
             raise
 
         self.LibType = type(self)        
-        self.FilePath = filepath    
+        self.FilePath = filepath 
+        self.ComponentCount = 0
+        self.Parts = []
 
         self._olefile = None
         self._olefile_open = False

src/pyaltiumlib/datatypes/__init__.py

@@ -44,5 +44,5 @@
     "PCBHoleShape",
     "PCBTextJustification",
     "PCBStrokeFont",
-    "PCBTextKind"
+    "PCBTextKind",
 ]

src/pyaltiumlib/datatypes/binaryreader.py

@@ -1,5 +1,9 @@
 from pyaltiumlib.datatypes.coordinate import Coordinate, CoordinatePoint
 
+# Configure logging
+import logging
+logger = logging.getLogger(__name__)
+
 class BinaryReader:
 
     def __init__(self, data):
@@ -12,7 +16,7 @@ def from_stream(cls, stream, size_length=4):
         data = stream.read( length )
 
         if len(data) != length:
-            raise ValueError("Stream does not match the declared block length.")        
+             logger.warning("Stream does not match the declared block length.")        
 
         return cls( data )       
 
@@ -25,39 +29,39 @@ def length(self):
     def read(self, length):
 
         if self.offset + length > len(self.data):
-            raise ValueError("Not enough data to read the requested length.")
+            logger.warning("Not enough data to read the requested length.")
 
         result = self.data[self.offset:self.offset + length]
         self.offset += length
         return result        
 
     def read_byte(self):
         if self.offset + 1 > len(self.data):
-            raise ValueError("Not enough data to read.")
+            logger.warning("Not enough data to read.")
         return self.read(1)
 
     def read_int8(self, signed=False):
         return int.from_bytes(self.read_byte(), signed=signed)
 
     def read_int16(self, signed=False):
         if self.offset + 2 > len(self.data):
-            raise ValueError("Not enough data to read an Int16.")
+            logger.warning("Not enough data to read an Int16.")
 
         value = int.from_bytes(self.data[self.offset:self.offset + 2], byteorder="little", signed=signed)
         self.offset += 2
         return value
 
     def read_int32(self, signed=False):
         if self.offset + 4 > len(self.data):
-            raise ValueError("Not enough data to read an Int32.")
+            logger.warning("Not enough data to read an Int32.")
 
         value = int.from_bytes(self.data[self.offset:self.offset + 4], byteorder="little", signed=signed)
         self.offset += 4
         return value
 
     def read_double(self):
         if self.offset + 8 > len(self.data):
-            raise ValueError("Not enough data to read a Double.")
+            logger.warning("Not enough data to read a Double.")
 
         raw_bytes = self.data[self.offset:self.offset + 8]
         self.offset += 8
@@ -69,7 +73,7 @@ def read_string_block(self, size_string=1):
         string_data = self.read( length_string )
 
         if len(string_data) != length_string:
-            raise ValueError("String does not match the declared string length.")        
+            logger.warning("String does not match the declared string length.")        
 
         return string_data.decode('windows-1252')
 
@@ -85,7 +89,7 @@ def read_unicode_text(self, length=32, encoding='utf-16-le'):
 
         while len(data) < length:
             if self.offset + 2 > len(self.data):
-                raise ValueError("Not enough data to read.")
+                logger.warning("Not enough data to read.")
 
             # Read 2 bytes (1 Unicode character)
             unicode_char = self.read(2)

src/pyaltiumlib/datatypes/svg_utils.py

@@ -0,0 +1,39 @@
+import math
+
+def get_arc_path(center, radius_x, radius_y, angle_start, angle_end):
+    """
+    Generate the SVG path data for the arc.
+
+    Args:
+        center (tuple): The center coordinates of the arc.
+        radius_x (int): The x-radius of the arc.
+        radius_y (int): The y-radius of the arc.
+
+    Returns:
+        str: The SVG path data for the arc.
+    """
+    def degrees_to_radians(degrees):
+        return (degrees * math.pi / 180) % (2*math.pi)
+    
+    angle_start = degrees_to_radians(angle_start)
+    angle_stop = degrees_to_radians(angle_end)
+    
+    if angle_start == angle_stop:
+        angle_stop -= 0.001
+    
+    start_x = center[0] + radius_x * math.cos(-angle_start)
+    start_y = center[1] + radius_y * math.sin(-angle_start)
+    end_x = center[0] + radius_x * math.cos(-angle_stop)
+    end_y = center[1] + radius_y * math.sin(-angle_stop)
+    
+    # Set large_arc_flag based on the angle difference
+    large_arc_flag = 1 if (angle_stop - angle_start) % (2 * math.pi) > math.pi else 0
+    
+    # Set sweep_flag to 0 for counterclockwise
+    sweep_flag = 0
+    
+    path_data = (
+        f"M {start_x},{start_y} "
+        f"A {radius_x},{radius_y} 0 {large_arc_flag},{sweep_flag} {end_x},{end_y}"
+    )
+    return path_data

src/pyaltiumlib/libcomponent.py

@@ -31,6 +31,12 @@ class LibComponent:
     """
     `string` that contains the description of the component
     """
+
+    Records = []
+    """
+    `List[any]` is a collection of records in their specific class 
+    contained in the library.
+    """
 
     def __init__(self, parent, name: str, description: str):
         """
@@ -39,12 +45,12 @@ def __init__(self, parent, name: str, description: str):
         """        
         self.LibFile = parent
         self.Name = name
-        self.Description = description
-                    
-        self.Records = [] 
-        self._BoundingBoxes = []
-        self._drawing_layer = {}
+        self.Description = description                  
+        self.Records = []
 
+        self._BoundingBoxes = []
+        self._graphic_layers = {}
+       
 
     def __repr__(self) -> Dict:
         """
@@ -84,7 +90,6 @@ def draw_svg(self, graphic, size_x: float, size_y: float, draw_bbox: bool = Fals
         :param float size_x: The width of the svgwrite drawing object
         :param float size_y: The height of the svgwrite drawing object        
         :param bool [optional] draw_bbox: Draw bounding boxes
-        :default draw_bbox: False
             
         :raises ImportError: If 'svgwrite' module is not installed.
         :raises ValueError: If 'graphic' is not the right instance.
@@ -102,7 +107,8 @@ def draw_svg(self, graphic, size_x: float, size_y: float, draw_bbox: bool = Fals
         validObj = []
         for obj in self.Records:
             if hasattr(obj, 'draw_svg') and callable(getattr(obj, 'draw_svg')):
-                validObj.append(obj)
+                if obj.is_initialized:
+                    validObj.append(obj)
             else:
                 logger.warning(f"Object: {obj} has no drawing function")
 
@@ -115,32 +121,25 @@ def draw_svg(self, graphic, size_x: float, size_y: float, draw_bbox: bool = Fals
                                  fill=self.LibFile._BackgroundColor.to_hex()))
 
         # Add Drawing Layer
+        self._graphic_layers = {}
         if hasattr(self, 'LibFile'):
-            if hasattr(self.LibFile, '_Layer'):
-                for x in sorted(self.LibFile._Layer, key=lambda obj: obj.drawing_order, reverse=True):
-                    self._drawing_layer[x.id] = graphic.add(graphic.g(id=x.svg_layer))
-                
+            if hasattr(self.LibFile, 'Layers'):
+                for x in sorted(self.LibFile.Layers, key=lambda obj: obj.drawing_order, reverse=True):
+                    self._graphic_layers[x.id] = graphic.add(graphic.g(id=x.svg_layer))
+                                
         if draw_bbox:
             for obj in validObj:
                     obj.draw_bounding_box( graphic, offset, zoom)
 
         # Draw Primitives
         for obj in validObj:
-            obj.draw_svg( graphic, offset, zoom)               
+            obj.draw_svg( graphic, offset, zoom)
+          
 
 
     def _autoscale(self, elements: List[Any], target_width: float, target_height: float, margin: float = 10.0) -> tuple:
         """
         Adjust the coordinates of elements to fit within the target dimensions using zoom.
-
-        Args:
-            elements (List[Any]): A list of objects with `get_bounding_box` method.
-            target_width (float): Target image width.
-            target_height (float): Target image height.
-            margin (float): Margin around the bounding box.
-
-        Returns:
-            tuple: (offset, zoom)
         """
 
         min_point = CoordinatePoint(Coordinate(float("inf")), Coordinate(float("inf")))

src/pyaltiumlib/pcblib/footprint.py

@@ -15,7 +15,7 @@ class PcbLibFootprint(LibComponent):
     
     :param class parent: reference to library file :class:`pyaltiumlib.schlib.lib.PCBLib`
     :param string name: name of the component
-    :param string description: description of the component
+    :param string [optional] description: description of the component
 
     :raises ValueError: If record id is not valid
     :raises ValueError: If component data can not be read
@@ -57,6 +57,9 @@ def _ReadFootprintData(self):
             if RecordID == 0:
                 StreamOnGoing = False
                 break
+
+            elif RecordID == 1:
+                self.Records.append( PcbArc(self, olestream) )
 
             elif RecordID == 2:
                 self.Records.append( PcbPad(self, olestream) )