harnice 0.3.0__tar.gz → 0.3.1__tar.gz

{harnice-0.3.0 → harnice-0.3.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: harnice
-Version: 0.3.0
+Version: 0.3.1
 Summary: Electrical System CAD
 Author-email: Kenyon Shutt <harnice.io@gmail.com>
 Project-URL: Documentation, https://harnice.io/
@@ -18,6 +18,7 @@ Requires-Dist: Pillow
 Requires-Dist: PyYAML
 Requires-Dist: xlwt
 Requires-Dist: webcolors
+Requires-Dist: prompt_toolkit
 Dynamic: license-file
 
 Harnice is a free, open source electrical system CAD tool. 

{harnice-0.3.0 → harnice-0.3.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "harnice"
-version = "0.3.0"
+version = "0.3.1"
 authors = [
   { name="Kenyon Shutt", email="harnice.io@gmail.com" },
 ]
@@ -19,7 +19,8 @@ dependencies = [
     "Pillow",
     "PyYAML",
     "xlwt",
-    "webcolors"
+    "webcolors",
+    "prompt_toolkit"
 ]
 
 [project.urls]

{harnice-0.3.0 → harnice-0.3.1}/src/harnice/fileio.py

@@ -209,12 +209,13 @@ def verify_revision_structure():
             f"Harnice only renders revisions with a blank status."
         )
 
-    print(f"Working on PN: {state.pn}, Rev: {state.rev}")
+    print(f"Rendering PN: {state.pn}, Rev: {state.rev}")
     rev_history.update_datemodified()
 
 
 def today():
-    return datetime.date.today().strftime("%-m/%-d/%y")
+    d = datetime.date.today()
+    return f"{d.month}/{d.day}/{d.year % 100}"
 
 
 def get_git_hash_of_harnice_src():

{harnice-0.3.0 → harnice-0.3.1}/src/harnice/lists/instances_list.py

@@ -5,23 +5,25 @@ from threading import Lock
 from harnice import fileio, state
 
 COLUMNS = [
-    "net", #documentation needed
-    "instance_name", #documentation needed
-    "print_name", #documentation needed
-    "bom_line_number", #documentation needed
-    "mfg", #documentation needed
-    "mpn",  # unique part identifier (manufacturer + part number concatenated)
+    "net", # the physical harness (represented by a net in Kicad) that this instance is part of
+    "instance_name", # the unique name of this instance
+    "print_name", # the non-unique, human-readable name of this instance, used for printing on output documents
+    "bom_line_number", # if this instance represents a physical procurable good, it gets assigned a line number on a bill of materials
+    "mfg", # manufacturer of this instance
+    "mpn",  # manufacturer part number
     "item_type",  # connector, backshell, whatever
     "parent_instance",  # general purpose reference
     "location_type",  # each instance is either better represented by one or ther other
     "segment_group",  # the group of segments that this instance is part of
     "segment_order",  # the sequential id of this item in its segment group
     "connector_group",  # a group of co-located parts (connectors, backshells, nodes)
-    "channel_group", #documentation needed
+    "channel_group", # other instances associated with this one because they are part of the same channel will share this value
     "circuit_id",  # which signal this component is electrically connected to
     "circuit_port_number",  # the sequential id of this item in its signal chain
     "node_at_end_a",  # derived from formboard definition
     "node_at_end_b",  # derived from formboard definition
+    "print_name_at_end_a", # human-readable name of this instance if needed, associated with 'node_at_end_a'
+    "print_name_at_end_b", # human-readable name of this instance if needed, associated with 'node_at_end_b'
     "parent_csys_instance_name",  # the other instance upon which this instance's location is based
     "parent_csys_outputcsys_name",  # the specific output coordinate system of the parent that this instance's location is based
     "translate_x",  # derived from parent_csys and parent_csys_name
@@ -29,9 +31,9 @@ COLUMNS = [
     "rotate_csys",  # derived from parent_csys and parent_csys_name
     "absolute_rotation",  # manual add, not nominally used unless it's a flagnote, segment, or node
     "csys_children",  # imported csys children from library attributes file
-    "cable_group", #documentation needed
-    "cable_container", #documentation needed
-    "cable_identifier", #documentation needed
+    "cable_group", # other instances associated with this one because they are part of the same cable will share this value
+    "cable_container", # which cable is this instance physically bundled inside of
+    "cable_identifier", # cable unique identifier
     "length",  # derived from formboard definition, the length of a segment
     "diameter",  # apparent diameter of a segment <---------- change to print_diameter
     "appearance",  # see harnice.utils.appearance for details
@@ -40,39 +42,39 @@ COLUMNS = [
     "note_parent",  # the instance the note applies to. typically don't use this in the instances list, just note_utils
     "note_text",  # the content of the note
     "note_affected_instances",  # list of instances that are affected by the note
-    "lib_repo", #documentation needed
-    "lib_subpath", #documentation needed
-    "lib_desc", #documentation needed
-    "lib_latest_rev", #documentation needed
-    "lib_rev_used_here", #documentation needed
-    "lib_status", #documentation needed
+    "lib_repo", # publically-traceable URL of the library this instance is from
+    "lib_subpath", # path to the instance within the library (directories between the product type and the part number)
+    "lib_desc", # description of the instance per the library's revision history
+    "lib_latest_rev", # the latest revision of the instance that exists in the remote library
+    "lib_rev_used_here", # the revision of the instance that is currently used in this project
+    "lib_status", # the status of the instance per the library's revision history
     "lib_releaseticket", #documentation needed
-    "lib_datestarted", #documentation needed
-    "lib_datemodified", #documentation needed
-    "lib_datereleased", #documentation needed
-    "lib_drawnby", #documentation needed
-    "lib_checkedby", #documentation needed
-    "project_editable_lib_modified", #documentation needed
-    "lib_build_notes", #documentation needed
-    "lib_tools", #documentation needed
+    "lib_datestarted", # the date this instance was first added to the library
+    "lib_datemodified", # the date this instance was last modified in the library
+    "lib_datereleased", # the date this instance was released in the library, if applicable, per the library's revision history
+    "lib_drawnby", # the name of the person who drew the instance, per the library's revision history
+    "lib_checkedby", # the name of the person who checked the instance, per the library's revision history
+    "project_editable_lib_modified", # a flag to indicate if the imported contents do not match the library's version (it's been locally modified)
+    "lib_build_notes", # recommended build notes that come with the instance from the library
+    "lib_tools", # recommended tools that come with the instance from the library
     "this_instance_mating_device_refdes",  # if connector, refdes of the device it plugs into
     "this_instance_mating_device_connector",  # if connector, name of the connector it plugs into
     "this_instance_mating_device_connector_mpn",  # if connector, mpn of the connector it plugs into
-    "this_net_from_device_refdes", #documentation needed
-    "this_net_from_device_channel_id", #documentation needed
-    "this_net_from_device_connector_name", #documentation needed
-    "this_net_to_device_refdes", #documentation needed
-    "this_net_to_device_channel_id", #documentation needed
-    "this_net_to_device_connector_name", #documentation needed
-    "this_channel_from_device_refdes",  # if channel, refdes of the device on one side of the channel
-    "this_channel_from_device_channel_id", #documentation needed
-    "this_channel_to_device_refdes",  # if channel, refdes of the device on the other side of the channel
-    "this_channel_to_device_channel_id", #documentation needed
-    "this_channel_from_channel_type", #documentation needed
-    "this_channel_to_channel_type", #documentation needed
-    "signal_of_channel_type", #documentation needed
-    "debug", #documentation needed
-    "debug_cutoff", #documentation needed
+    "this_net_from_device_refdes", # if this instance is a channel, circuit, conductor, etc, the refdes of the device it interfaces with, just within this net
+    "this_net_from_device_channel_id", # if this instance is a channel, circuit, conductor, etc, the channel id in the device it interfaces with, just within this net
+    "this_net_from_device_connector_name", # if this instance is a channel, circuit, conductor, etc, the name of the connector it interfaces with, just within this net
+    "this_net_to_device_refdes", # if this instance is a channel, circuit, conductor, etc, the refdes of the device it plugs into just within this net
+    "this_net_to_device_channel_id", # if this instance is a channel, circuit, conductor, etc, the channel id in the device it plugs into, just within this net
+    "this_net_to_device_connector_name", # if this instance is a channel, circuit, conductor, etc, the name of the connector it plugs into, just within this net
+    "this_channel_from_device_refdes",  # if this instance is a channel, circuit, conductor, etc, the refdes of the device it interfaces with, at the very end of the channel
+    "this_channel_from_device_channel_id", # if this instance is a channel, circuit, conductor, etc, the channel id in the device it interfaces with, at the very end of the channel
+    "this_channel_to_device_refdes",  # if this instance is a channel, circuit, conductor, etc, the refdes of the device it plugs into, at the very end of the channel
+    "this_channel_to_device_channel_id", # if this instance is a channel, circuit, conductor, etc, the channel id in the device it plugs into, at the very end of the channel
+    "this_channel_from_channel_type", # if this instance is a channel, circuit, conductor, etc, the type of the channel it interfaces with, at the very end of the channel
+    "this_channel_to_channel_type", # if this instance is a channel, circuit, conductor, etc, the type of the channel it plugs into, at the very end of the channel
+    "signal_of_channel_type", # if this instance is a channel, circuit, conductor, etc, the signal of the channel it interfaces with, at the very end of the channel
+    "debug", # the call chain of the function that last modified this instance row
+    "debug_cutoff", # blank cell to visually cut off the previous column
 ]
 
 

{harnice-0.3.0 → harnice-0.3.1}/src/harnice/lists/rev_history.py

@@ -6,27 +6,59 @@ from harnice import fileio, state
 
 # === Global Columns Definition ===
 COLUMNS = [
-    "product", #documentation needed
-    "mfg", #documentation needed
-    "pn", #documentation needed
-    "desc", #documentation needed
-    "rev", #documentation needed
-    "status", #documentation needed
-    "releaseticket", #documentation needed
-    "library_repo", #documentation needed
-    "library_subpath", #documentation needed
-    "datestarted", #documentation needed
-    "datemodified", #documentation needed
-    "datereleased", #documentation needed
-    "git_hash_of_harnice_src", #documentation needed
-    "drawnby", #documentation needed
-    "checkedby", #documentation needed
-    "revisionupdates", #documentation needed
-    "affectedinstances", #documentation needed
+    "product",  # the harnice product type (e.g. "harness", "connector", "device", "system", "macro", "flagnote", "tblock")
+    "mfg",  # who manufactures this product (blank ok)
+    "pn",  # name, part number, other identifier of this part. mfg+mpn combination must be unique within the library.
+    "desc",  # a brief description of this product
+    "rev",  # the revision of the part
+    "status",  # "released", "obsolete", etc. Harnice will not render a revision if the status has text in this field as a form of protection.
+    "releaseticket",  # many companies do this, but it's not required.
+    "library_repo",  # auto-filled on render if the current working directory is discovered to be a library repository.
+    "library_subpath",  # auto-filled on render if in a library repository, this is the chain of directories between the product type and the part number
+    "datestarted",  # auto-filled to be the date when this part was first intialized
+    "datemodified",  # updates to today's date upon rendering
+    "datereleased",  # up to user to fill in as needed
+    "git_hash_of_harnice_src",  # auto-filled, git hash of the harnice source code during the latest render
+    "drawnby",  # auto-filled, the person who created the part
+    "checkedby",  # the person who checked the part, blank ok
+    "revisionupdates",  # a brief description of the changes made to this revision
+    "affectedinstances",  # the instance names of the instances that were affected by this revision. can be referenced later by PDF builders and more.
 ]
 
 
 def overwrite(content_dict):
+    """
+Overwrite a revision history entry.
+
+**Arguments:**
+
+ - `content_dict` (dict): The content to overwrite the revision history entry with.
+    - This should be a dictionary with the keys and values to overwrite.
+    - The keys should be the column names, and the values should be the new values.
+    - Some keys are protected and cannot be overwritten:
+        - `"product"`
+        - `"mfg"`
+        - `"pn"`
+        - `"rev"`
+        - `"releaseticket"`
+        - `"library_repo"`
+        - `"library_subpath"`
+        - `"datestarted"`
+
+The function will update the revision history file as referenced by the current product file structure.
+
+**Returns:**
+
+ - `None`
+
+**Raises:**
+
+ - `KeyError`: If a key is provided that is not in the COLUMNS list.
+ - `KeyError`: If a protected key is provided.
+ - `ValueError`: If the revision history file is not found.
+ - `ValueError`: If the revision is not found in the revision history file.
+ - `RuntimeError`: If `state.rev` is not set.
+    """
     PROTECTED_KEYS = [
         "product",
         "mfg",
@@ -84,6 +116,31 @@ def overwrite(content_dict):
 
 
 def info(rev=None, path=None, field=None, all=False):
+    """
+    Get information about a revision history entry.
+
+    **Arguments:**
+
+    - `rev` (str): The revision to get information about.
+    - `path` (str): The path to the revision history file.
+        - If not provided, the function will use the default path: `"revision history"`.
+    - `field` (str): The field to get information about.
+        - If not provided, the function will return the entire row.
+        - If provided, the function will return the value of the field.
+    - `all` (bool): If `True`, return all rows.
+        - If not provided, the function will return the first row.
+
+    **Returns:**
+
+    - `dict`: The row of the revision history entry (when `field` is not provided).
+    - `list`: A list of all rows in the revision history file (when `all=True`).
+    - `str`: The value of the field (when `field` is provided).
+
+    **Raises:**
+
+    - `FileNotFoundError`: If the revision history file is not found.
+    - `ValueError`: If the revision is not found in the revision history file.
+    """
     if path is None:
         path = fileio.path("revision history")
 
@@ -104,7 +161,6 @@ def info(rev=None, path=None, field=None, all=False):
 
     for row in rows:
         if row.get("rev") == rev:
-
             # ------------------------------------------------------
             # Field requested
             # ------------------------------------------------------
@@ -142,6 +198,17 @@ def info(rev=None, path=None, field=None, all=False):
 
 
 def initial_release_exists():
+    """
+    Check if an initial release exists.
+
+    **Arguments:**
+
+    None
+
+    **Returns:**
+
+    - `bool`: `True` if a revision with the text `"INITIAL RELEASE"` in the `"revisionupdates"` field exists, `False` otherwise.
+    """
     try:
         for row in fileio.read_tsv("revision history"):
             if str(row.get("revisionupdates", "")).strip() == "INITIAL RELEASE":
@@ -153,12 +220,39 @@ def initial_release_exists():
 
 
 def initial_release_desc():
+    """
+    Get the description of the initial release.
+
+    **Arguments:**
+
+    None
+
+    **Returns:**
+
+    - `str`: The description of the revision which has `revisionupdates == 'INITIAL RELEASE'`.
+    """
     for row in fileio.read_tsv("revision history"):
         if row.get("revisionupdates") == "INITIAL RELEASE":
             return row.get("desc")
 
 
 def update_datemodified():
+    """
+    Update the `datemodified` field of the current revision with today's date.
+
+    **Arguments:**
+
+    None
+
+    **Returns:**
+
+    - `None`
+
+    **Raises:**
+
+    - `ValueError`: If the revision history file is not found.
+    - `ValueError`: If the revision is not found in the revision history file.
+    """
     target_rev = state.partnumber("R")
 
     # Read all rows
@@ -182,42 +276,94 @@ def update_datemodified():
         writer.writerows(rows)
 
 
-def new():
+def new(ignore_product=False, path=None):
+    """
+    Create a new revision history file.
+
+    **Arguments:**
+
+    - `ignore_product` (bool):
+        - If `True`, the function will raise an error if `state.product` is not set first.
+        - If `False`, the function will prompt the user to select a product type.
+
+    **Returns:**
+
+    - `None`
+
+    **Raises:**
+
+    - `ValueError`: If attempting to create a new revision history file without a product type when `ignore_product=True`.
+    - `ValueError`: If attempting to overwrite an existing revision history file.
+    """
     columns = COLUMNS
-    from harnice.cli import select_product_type
 
-    global product
-    product = select_product_type()
-    with open(fileio.path("revision history"), "w", newline="", encoding="utf-8") as f:
-        writer = csv.DictWriter(f, fieldnames=columns, delimiter="\t")
-        writer.writeheader()
+    if path is None:
+        path = fileio.path("revision history")
+
+    if not ignore_product:
+        from harnice.cli import select_product_type
+
+        state.set_product(select_product_type())
+
+    if ignore_product and not state.product:
+        raise ValueError(
+            "You tried to create a new revision history file without a product type. This is not allowed."
+        )
+
+    if not os.path.exists(path):
+        with open(path, "w", newline="", encoding="utf-8") as f:
+            writer = csv.DictWriter(f, fieldnames=columns, delimiter="\t")
+            writer.writeheader()
+
+    else:
+        raise ValueError(
+            "You tried to overwrite a revision history file- this is not allowed."
+        )
 
 
 def append(next_rev=None):
-    from harnice import cli
+    """
+    Append a new revision history entry to the current revision history file.
+
+    If the revision history file does not exist, the function will create it.
+    If the revision history file exists, the function will append a new entry to the file.
+
+    It will prompt the user for the following fields:
+
+    - `product`: The product type of the part.
+    - `desc`: The description of the part.
+    - `revisionupdates`: What is the purpose of this revision?
+
+    If the previous revision has a blank status, the function will prompt the user to obsolete it with a message.
+
+    **Arguments:**
+
+    - `next_rev` The next revision number to append.
+
+    **Returns:**
 
-    global product
+    - `None`
+    """
+    from harnice import cli
 
     if not os.path.exists(fileio.path("revision history")):
         new()
     rows = fileio.read_tsv("revision history")
-    product_name = None
     if rows:
         for row in reversed(rows):
             candidate = (row.get("product") or "").strip()
             if candidate:
-                product_name = candidate
+                state.set_product(candidate)
                 break
-    if not product_name:
-        product_name = globals().get("product")
-    if not product_name:
-        product_name = cli.select_product_type()
-    product = product_name
+    if not state.product:
+        state.set_product(cli.select_product_type())
 
     default_desc = ""
-    if product_name:
+    if state.product:
         try:
-            product_module = importlib.import_module(f"harnice.products.{product_name}")
+            product_module = importlib.import_module(
+                f"harnice.products.{state.product}"
+            )
         except ModuleNotFoundError:
             product_module = None
         else:
@@ -252,7 +398,7 @@ def append(next_rev=None):
 
     if desc in [None, ""]:
         desc = cli.prompt(
-            f"Enter a description of this {product_name}",
+            f"Enter a description of this {state.product}",
             default=default_desc,
         )
 
@@ -305,7 +451,7 @@ def append(next_rev=None):
 
     rows.append(
         {
-            "product": product_name,
+            "product": state.product,
             "pn": state.pn,
             "rev": next_rev,
             "desc": desc,
@@ -323,3 +469,67 @@ def append(next_rev=None):
         writer = csv.DictWriter(f, fieldnames=columns, delimiter="\t")
         writer.writeheader()
         writer.writerows(rows)
+
+
+def part_family_append(content_dict, rev_history_path):
+    """
+    Append a new revision history entry to the part family revision history file.
+
+    Intended to be called by part family scripts only.
+
+    The function will automatically update the following fields in the content dictionary:
+
+    - `datemodified`: Set to today's date
+    - `drawnby`: Set to the current user's name
+    - `git_hash_of_harnice_src`: Set to the current git hash of the harnice source code
+
+    If the revision history file does not exist, the function will create it.
+    If an entry with the same revision number already exists, the function will update that entry.
+    Otherwise, the function will append a new entry to the file.
+
+    **Arguments:**
+
+    - `content_dict` (dict): The content to append to the part family revision history file.
+        - Should contain keys matching the `COLUMNS` list.
+        - The `rev` key is used to determine if an entry already exists.
+    - `rev_history_path` (str): The path to the part family revision history file.
+
+    **Returns:**
+
+    - `None`
+
+    **Raises:**
+
+    - `ValueError`: If the content dictionary contains invalid keys or missing required fields.
+    """
+    actual_content_dict = content_dict
+
+    actual_content_dict["datemodified"] = fileio.today()
+    actual_content_dict["drawnby"] = fileio.drawnby()["name"]
+    actual_content_dict["git_hash_of_harnice_src"] = (
+        fileio.get_git_hash_of_harnice_src()
+    )
+
+    rev = content_dict.get("rev")
+
+    if os.path.exists(rev_history_path):
+        rows = fileio.read_tsv(rev_history_path)
+    else:
+        rows = []
+
+    found = False
+    for i, row in enumerate(rows):
+        if row.get("rev") == actual_content_dict.get("rev"):
+            rows[i] = actual_content_dict
+            found = True
+            break
+
+    if not found:
+        rows.append(actual_content_dict)
+
+    if not os.path.exists(rev_history_path):
+        new(path=rev_history_path, ignore_product=True)
+    with open(rev_history_path, "w", newline="", encoding="utf-8") as f:
+        writer = csv.DictWriter(f, fieldnames=COLUMNS, delimiter="\t")
+        writer.writeheader()
+        writer.writerows(rows)

{harnice-0.3.0 → harnice-0.3.1}/src/harnice/products/harness.py

@@ -162,7 +162,7 @@ for instance in instances:
             if instance.get("item_type") == "circuit":
                 circuit_instance = instance
                 connector_at_end_a = instances_list.attribute_of(instance.get("node_at_end_a"), "connector_group")
-new_instance_name = f"{{circuit_instance.get("instance_name")}}-special_contact"
+new_instance_name = f"{{circuit_instance.get('instance_name')}}-special_contact"
 circuit_id = int(circuit_instance.get("circuit_id"))
 instances_list.new_instance(
     new_instance_name, {{
@@ -181,7 +181,7 @@ circuit_utils.squeeze_instance_between_ports_in_circuit(
 # example: add a backshell
 for instance in instances:
     if instance.get("instance_name") in ["X1.B.conn", "PREAMP2.in2.conn"]:
-        instances_list.new_instance(f"{{instance.get("connector_group")}}.bs", {{
+        instances_list.new_instance(f"{{instance.get('connector_group')}}.bs", {{
             "bom_line_number": True,
             "mpn": "M85049-90_9Z03",
             "item_type": "backshell",
@@ -193,7 +193,7 @@ for instance in instances:
             "lib_repo": "https://github.com/harnice/harnice"
         }})
         instances_list.modify(instance.get("instance_name"), {{
-            "parent_csys_instance_name": f"{{instance.get("connector_group")}}.bs",
+            "parent_csys_instance_name": f"{{instance.get('connector_group')}}.bs",
             "parent_csys_outputcsys_name": "connector",
         }})
 
@@ -270,23 +270,23 @@ for x in range(2):
 
         elif instance.get("item_type") in ["conductor", "conductor-segment"]:
             instances_list.modify(instance.get("instance_name"), {{
-                "print_name": f"'{{instance.get("cable_identifier")}}' of '{{instances_list.attribute_of(instance.get("cable_group"), "print_name")}}'"
+                "print_name": f"'{{instance.get('cable_identifier')}}' of '{{instances_list.attribute_of(instance.get('cable_group'), 'print_name')}}'"
             }})
 
         elif instance.get("item_type") == "net-channel":
-            print_name = f"'{{instance.get("this_channel_from_device_channel_id")}}' of '{{instance.get("this_channel_from_device_refdes")}}' to '{{instance.get("this_channel_to_device_channel_id")}}' of '{{instance.get("this_channel_to_device_refdes")}}'"
+            print_name = f"'{{instance.get('this_channel_from_device_channel_id')}}' of '{{instance.get('this_channel_from_device_refdes')}}' to '{{instance.get('this_channel_to_device_channel_id')}}' of '{{instance.get('this_channel_to_device_refdes')}}'"
             instances_list.modify(instance.get("instance_name"), {{"print_name": print_name}})
 
         elif instance.get("item_type") == "net-channel-segment":
-            print_name = f"'{{instances_list.attribute_of(instance.get("parent_instance"), "this_channel_from_device_channel_id")}}' of '{{instances_list.attribute_of(instance.get("parent_instance"), "this_channel_from_device_refdes")}}' to '{{instances_list.attribute_of(instance.get("parent_instance"), "this_channel_to_device_channel_id")}}' of '{{instances_list.attribute_of(instance.get("parent_instance"), "this_channel_to_device_refdes")}}'"
+            print_name = f"'{{instances_list.attribute_of(instance.get('parent_instance'), 'this_channel_from_device_channel_id')}}' of '{{instances_list.attribute_of(instance.get('parent_instance'), 'this_channel_from_device_refdes')}}' to '{{instances_list.attribute_of(instance.get('parent_instance'), 'this_channel_to_device_channel_id')}}' of '{{instances_list.attribute_of(instance.get('parent_instance'), 'this_channel_to_device_refdes')}}'"
             instances_list.modify(instance.get("instance_name"), {{"print_name": print_name}})
 
         elif instance.get("item_type") == "connector":
-            print_name = f"{{instance.get("connector_group")}}"
+            print_name = f"{{instance.get('connector_group')}}"
             instances_list.modify(instance.get("instance_name"), {{"print_name": print_name}})
 
         elif instance.get("item_type") == "cable-segment":
-            print_name = f"{{instance.get("cable_group")}}"
+            print_name = f"{{instance.get('cable_group')}}"
             instances_list.modify(instance.get("instance_name"), {{"print_name": print_name}})
 
         elif instance.get("item_type") == "contact":

{harnice-0.3.0 → harnice-0.3.1}/src/harnice/products/part.py

@@ -635,6 +635,4 @@ def render():
     png_path = fileio.path("drawing png")
     img.save(png_path, dpi=(1000, 1000))
 
-    print()
-    print(f"Part file '{state.partnumber('pn')}' updated")
-    print()
+    print(f"Part file '{state.partnumber('pn')}' updated\n")