[rasterio] `transform.rowcol` return typing possibly incorrect?
Offen
Anfängerfreundlich
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 5.1k
- Forks
- 2.1k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 82
Beschreibung
From what we expect and what the docstring of this method says in rasterio, I believe the return typing isn't strictly int it can be floats? Please could someone double check to verify?
https://github.com/rasterio/rasterio/blob/main/rasterio/transform.py#L295
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, rasterio/transform.py ungefähr ab Zeile 295 zu lesen, und vergleiche den Docstring von rowcol mit seinem Laufzeitverhalten. Suche dann das entsprechende typeshed-Stub und dessen Tests, prüfe, ob Ergebnisse vom Typ float möglich sind, und aktualisiere bei Bedarf den deklarierten Rückgabetyp, damit das Stub der Implementierung entspricht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 68/100