mirror of
https://github.com/IfcOpenShell/IfcOpenShell.git
synced 2026-08-09 17:31:45 +00:00
Fix all remaining ty type-check failures on ci-lint
The ci-lint workflow's ty steps fail on every branch because base v0.8.0 has four diagnostics. ty check (bonsai): - root/operator.py: bpy.data.objects.get() can return None, so UnlinkObject._execute could put None in its objects list and crash on the first attribute access when an unknown object name is passed. Handle the miss explicitly, which also satisfies the declared list[bpy.types.Object] type. - tool/sequence.py: ty does not narrow Literal types through membership tests on list literals, so the assert_never() exhaustive check was flagged. Use tuple literals, which ty narrows, keeping the exhaustiveness check intact. ty check (ios): - draw.py: arrange_polygons was called through conditional argument splats that let the same call site work against pre-April-2026 wrappers lacking arrange_polygon_settings and the logger parameter. No runtime bug for current builds, but the dynamic splats cannot be typed against the fixed 3-parameter signature. Drop the old-build workaround and call the current signature directly, following the precedent of3d8115ebc5which dropped similar old-build workarounds in ifcopenshell.file. Verified against a current wrapper build that the direct call arranges polygons and serializes to SVG, with and without a logger. - Optimise.py: igraph is an optional dependency with a guarded import and a toposort fallback, but it was missing from the ios type-check venv so ty could not resolve it. Add it to type-check-requirements next to the toposort fallback that is already listed. After this, poe ty-bonsai and poe ty-ios both pass cleanly. Generated with the assistance of an AI coding tool. (cherry picked from commit6d6d92b849)
This commit is contained in:
committed by
Dion Moult
parent
5d13791a01
commit
070b7793f4
@@ -413,12 +413,13 @@ class UnlinkObject(bpy.types.Operator, tool.Ifc.Operator):
|
||||
skip_invoke: bpy.props.BoolProperty(default=False, options={"SKIP_SAVE"})
|
||||
|
||||
def _execute(self, context):
|
||||
objects: list[bpy.types.Object]
|
||||
if self.obj:
|
||||
objects = [bpy.data.objects.get(self.obj)]
|
||||
requested_obj = bpy.data.objects.get(self.obj)
|
||||
objects = [requested_obj] if requested_obj is not None else []
|
||||
else:
|
||||
objects = context.selected_objects
|
||||
|
||||
objects: list[bpy.types.Object]
|
||||
for obj in objects:
|
||||
was_active_object = obj == context.active_object
|
||||
|
||||
|
||||
@@ -1158,11 +1158,11 @@ class Sequence(bonsai.core.tool.Sequence):
|
||||
props.task_input_colors.clear()
|
||||
for group, data in groups.items():
|
||||
for predefined_type in data["PredefinedType"]:
|
||||
if group in ["CREATION", "OPERATION", "MOVEMENT_TO"]:
|
||||
if group in ("CREATION", "OPERATION", "MOVEMENT_TO"):
|
||||
predefined_type_item = props.task_output_colors.add()
|
||||
elif group in ["MOVEMENT_FROM"]:
|
||||
elif group in ("MOVEMENT_FROM",):
|
||||
predefined_type_item = props.task_input_colors.add()
|
||||
elif group in ["USERDEFINED", "DESTRUCTION"]:
|
||||
elif group in ("USERDEFINED", "DESTRUCTION"):
|
||||
predefined_type_item = props.task_input_colors.add()
|
||||
predefined_type_item2 = props.task_output_colors.add()
|
||||
predefined_type_item2.name = predefined_type
|
||||
|
||||
@@ -3,6 +3,7 @@ cjio >=0.8, <0.10
|
||||
deepdiff
|
||||
docutils
|
||||
flask
|
||||
igraph
|
||||
isodate
|
||||
jinja2
|
||||
lark
|
||||
|
||||
Reference in New Issue
Block a user