AllenCell / AllenCell/allencell-segmenter-ml

returns optional pattern

Offen
#522 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
refactor
Vorherrschende Sprache
Python
Sterne
5
Forks
3
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Our practice of doing null checks on fields returned by getters whose value can be null (in order to satify `mypy` linter) really feels like an anti pattern, especially when we handle null values by throwing an error

https://github.com/AllenCell/allencell-segmenter-ml/pull/519#discussion_r1779307204

Could the error just be thrown internal to the getter, for violating an invariant?

We need research into how type checking in Python should be done.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.