outer_boundary_corners and __contains__ raise ValueError on some areas
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Riproduci l’esempio usando pyresample/geometry.py e analizza il modo in cui outer_boundary_corners produce le coordinate prima che vengano passate a pyresample/spherical_geometry.py:50. Confronta questo percorso con il controllo di contenimento dell’area. Il lavoro è completato quando un input valido di longitudine/latitudine restituisce True o False da contains, e il comportamento previsto per gli angoli del limite esterno non validi è stato stabilito.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Checking the outer boundary corners of some areas fails with ValueError. In this case, checking if a certain point is contained also fails.
Code Sample, a minimal, complete, and verifiable piece of code
from pyresample.area_config import parse_area_file
area = parse_area_file(
"/data/gholl/checkouts/satpy/satpy/etc/areas.yaml",
"seviri_0deg")[0]
print((0, 0) in area)
print(area.outer_boundary_corners)
Problem description
raises ValueError, probably because the Earth is round and has no corners.
Expected Output
For the outer_boundary_corners, maybe NaN would be appropriate, or maybe a ValueError is actually correct. However, I don't think (lon, lat) in area should raise a ValueError for any valid lon/lat (see also #268). For valid input, this should always return either True or False.
Actual Result, Traceback if applicable
Traceback (most recent call last):
File "/data/gholl/checkouts/protocode/mwe/outer-corners.py", line 5, in <module>
print(area.outer_boundary_corners)
File "/data/gholl/checkouts/pyresample/pyresample/geometry.py", line 2134, in outer_boundary_corners
return [Coordinate(corner_lons[0], corner_lats[0]),
File "/data/gholl/checkouts/pyresample/pyresample/spherical_geometry.py", line 50, in __init__
raise ValueError('Illegal (lon, lat) coordinates: (%s, %s)'
ValueError: Illegal (lon, lat) coordinates: (inf, inf)
Versions of Python, package at hand and relevant dependencies
pyresample v1.22.0-8-gee7c1b9
- Lingua principale
- Python
- Stelle
- 388
- Fork
- 103
- Merge medio
- 5g 17h
- PR unite (30g)
- 5
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di pytroll/pyresample
-
Geometry `__eq__` and `__hash__` violate Python's hash/equality contractForse di nuovo libera @pnuu l’ha presa 46 giorni fa e non c’è nessuna pull request aperta. Apertabug Priority-Medium
pytroll/pyresample#745 · 1 commento · 3 assegnatari ·
-
Improve resampler x/y coordinate metadataForse di nuovo libera @djhoese l’ha presa 64 giorni fa e non c’è nessuna pull request aperta. Apertabug
pytroll/pyresample#739 · 1 assegnatario ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
pytroll/pyresample#698 · 1 commento ·
-
`AreaDefinition.get_area_slices` doesn't work for cropped geostationary areas like SEVIRI RSSAperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
pytroll/pyresample#694 · 5 commenti ·
-
gradient_search comb effectForse di nuovo libera @mraspaud l’ha presa 385 giorni fa e non c’è nessuna pull request aperta. Aperta
pytroll/pyresample#682 · 7 commenti · 1 assegnatario ·
Tutte le issue di pytroll/pyresample
Issue simili
-
[Bug]: Revit selector reads process ids 2000-2099 as years (flaky AgentClientTests.AYearOrAProcessIdSelectsTheHost)Forse già presa @Wurschdhaud l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
pyrevitlabs/pyRevit#3817 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
521xueweihan/HelloGitHub#4074 ·
-
follow-up
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
Helm chart: allow setting ingressClassName on the IngressForse già presa @commarla l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
pgadmin-org/pgadmin4#10658 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100