Confirm outdated review comment on _parse_network_input is no longer applicable

[Copilot]

• Run mypy and identify all type errors
• Fix NodeObservation.node return type to Optional[int]
• Fix NetworkObservation.nodes list comprehension to filter None values
• Add None guard in NodeModelResult.node before calling .item()
• Remove SpatialField inheritance from NetworkModelResult (incompatible extract() signature)
• Fix matching.py with proper cast() calls and explicit raw_mod_data type
• Remove NetworkModelResult from raw_mod_data type union in Comparer (after extraction it becomes NodeModelResult)
• Fix _to_model() return type to include NodeModelResult
• Fix load() method type annotation in Comparer
• All 42 mypy source files pass with no issues

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[jpalm3r]

@copilot run mypy and solve the errors

[Copilot]

@copilot run mypy and solve the errors

Fixed in commit 5550ee8. Resolved all 19 mypy errors:

• NodeObservation.node return type changed to Optional[int]
• Added None guard in NodeModelResult.node before calling .item()
• Removed SpatialField inheritance from NetworkModelResult (its extract() signature is incompatible with the Protocol)
• Fixed matching.py with proper cast() calls and explicit raw_mod_data type annotation
• Removed NetworkModelResult from raw_mod_data type union in Comparer (after extraction it becomes NodeModelResult, never stored as NetworkModelResult)
• Fixed _to_model() return type and load() type annotation

mypy src/modelskill/ now reports: Success: no issues found in 42 source files

[jpalm3r]

Closed since it is obsolete. The issues here were solved in #586