Stop storing a figure that was never built, and reach the floor #370 sets

[alexarje]

Continuing the tail of #350. mypy 34 → 26, 682 tests pass. This is where the remaining count stops being a matter of effort.

Four pose methods stored None as though it were a figure

Since the render helpers gained honest return types, self.pose_waterfall_figure = mgf could write None into an attribute declared to hold an MgFigure. The assignment is conditional now: a figure that could not be built leaves the attribute absent, which is what the declared-attribute model means and what keeps hasattr truthful. A stored None would have been found by show() and then failed on it.

Three lazily built things behind a flag mypy will not correlate

The GPU Farneback object is created under _use_gpu and used under _use_gpu; the two Eulerian lowpass pyramids are built together on the first frame and used together after. Asserted at use, annotated at declaration.

The floor

Nine of the twenty-six that remain are inside motion_mp(), which raises on its first call — #370. Typing code that cannot run would be work spent hiding a breakage, so those are untouched.

That leaves 17 addressable, and it means zero is not reachable while #370 stands. Whoever takes || true off the CI step is choosing between repairing that function, retiring it, and allowlisting the one file.

Checked by hand that both Eulerian modes still run, including the motion mode that uses those pyramids.

🤖 Generated with Claude Code