Skip to content

venv for mypy and a few small fixex#1682

Merged
Christian-B merged 5 commits into
masterfrom
mypy_fix
Jul 21, 2026
Merged

venv for mypy and a few small fixex#1682
Christian-B merged 5 commits into
masterfrom
mypy_fix

Conversation

@Christian-B

Copy link
Copy Markdown
Member

some small errors and fixes/ignores

spynnaker/pyNN/models/defaults.py:193: error: "type[object]" has no attribute "default_parameters" [attr-defined]

  • method should be removed so ignore

spynnaker/pyNN/utilities/neo_csv.py:33: error: Module "_csv" has no attribute "_writer"; maybe "writer" or "Writer"? [attr-defined]
_writer and reader where a temp thing (in csv code) for 3.9

  • use Writer and Reader

spynnaker/pyNN/models/populations/population.py:687: error: Incompatible types in assignment (expression has type "AbstractPyNNModel | PopulationApplicationVertex", variable has type "AbstractPyNNModel") [assignment]

  • Define the types so mypy does not assume just one

@Christian-B
Christian-B requested a review from rowleya July 20, 2026 15:27
@Christian-B
Christian-B merged commit b640dd3 into master Jul 21, 2026
16 checks passed
@Christian-B
Christian-B deleted the mypy_fix branch July 21, 2026 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants