warehack.ing/WireViz
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use the same lambda in get_bom_index() as for deduplicating BOM

Browse Source 
Move the lambda declaration out of the function scope for common
access from two different functions.
This commit is contained in:
KV 2020-11-14 20:51:50 +01:00
parent 6525537312
commit 45b13ef797
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/wireviz/wv_bom.py
View File

@ -36,6 +36,8 @@ def get_additional_component_bom(component: Union[Connector, Cable]) -> List[dic
})
return(bom_entries)
bom_types_group = lambda bt: (bt['item'], bt['unit'], bt['manufacturer'], bt['mpn'], bt['pn'])
def generate_bom(harness):
from wireviz.Harness import Harness # Local import to avoid circular imports
bom_entries = []
@ -97,7 +99,6 @@ def generate_bom(harness):
# deduplicate bom
bom = []
bom_types_group = lambda bt: (bt['item'], bt['unit'], bt['manufacturer'], bt['mpn'], bt['pn'])
for group in Counter([bom_types_group(v) for v in bom_entries]):
group_entries = [v for v in bom_entries if bom_types_group(v) == group]
designators = []
@ -121,7 +122,7 @@ def get_bom_index(bom: List[dict], extra: AdditionalComponent) -> int:
# Remove linebreaks and clean whitespace of values in search
target = tuple(clean_whitespace(v) for v in (extra.description, extra.unit, extra.manufacturer, extra.mpn, extra.pn))
for entry in bom:
if (entry['item'], entry['unit'], entry['manufacturer'], entry['mpn'], entry['pn']) == target:
if bom_types_group(entry) == target:
return entry['id']
return None