From 6c4200e36c337e9b355f4d804c65de3a588c7056 Mon Sep 17 00:00:00 2001 From: KV Date: Sat, 4 Jul 2020 23:46:36 +0200 Subject: [PATCH] Correct a wrong TODO statement --- src/wireviz/Harness.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/wireviz/Harness.py b/src/wireviz/Harness.py index eba63e2..366f89e 100644 --- a/src/wireviz/Harness.py +++ b/src/wireviz/Harness.py @@ -325,7 +325,7 @@ class Harness: bom.extend(bom_connectors) # cables # TODO: If category can have other non-empty values than 'bundle', maybe it should be part of item name? - # Otherwise, it can be removed from the cable_group because it will allways be empty. + # The category needs to be included in cable_group to keep the bundles excluded. cable_group = lambda c: (c.category, c.type, c.gauge, c.gauge_unit, c.wirecount, c.shield, c.manufacturer, c.manufacturer_part_number, c.internal_part_number) groups = Counter([cable_group(v) for v in self.cables.values() if v.category != 'bundle']) for group in groups: