Update deps 2024 12 06 #207
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
| if isinstance(whole, list): | ||
| return [] |
There was a problem hiding this comment.
Für was ist das? Eher error?
There was a problem hiding this comment.
Aber sollte man dann nicht einen Fehler werfen? Leere Rückgabe ist ja irreführend.
| @@ -449,11 +450,11 @@ def test_ndvi_xarray(self): | |||
| expected.coords), | |||
| msg=f'{array.coords} \n!=\n {expected.coords}') | |||
|
|
|||
| # TODO: Test with np.array_equal(? | |||
There was a problem hiding this comment.
Wieso geht das nicht mehr mit array equal?
There was a problem hiding this comment.
Mir scheint, dass OWSLib in verschiedenen Versionen verschiedene Felder in der Response hat. Deshalb kann man nicht mit equals testen. Ich teste hier auf das Subset welches wir erwarten. Das TODO kann man vermutlich rausnehmen.
There was a problem hiding this comment.
Kann man das dann hier kurz erklären?
| @@ -19,36 +19,36 @@ packages = find: | |||
| python_requires = >=3.9 | |||
There was a problem hiding this comment.
Du hast alle Notebooks geändert. Manche haben z.B. ein RESPECT-Volume ausgegeben. Das könnte man clean laufen lassen. Davon abgesehen. Müssen alle geändert werden?
There was a problem hiding this comment.
Nicht ganz. Es gibt zwei Notebooks die sind immernoch kaputt. Bei vielen war irgendwas veraltet, funktionierte nicht oder es waren unerwünschte Ausgaben z.b. Warnings drin.
There was a problem hiding this comment.
Aber das RESPECT-Volume gibt es ja nicht normalerweise.
Vielleicht kannst du das mit dem neuen Testkram ausprobieren?
| if isinstance(whole, list): | ||
| return [] |
There was a problem hiding this comment.
Aber sollte man dann nicht einen Fehler werfen? Leere Rückgabe ist ja irreführend.
| @@ -449,11 +450,11 @@ def test_ndvi_xarray(self): | |||
| expected.coords), | |||
| msg=f'{array.coords} \n!=\n {expected.coords}') | |||
|
|
|||
| # TODO: Test with np.array_equal(? | |||
There was a problem hiding this comment.
Kann man das dann hier kurz erklären?
| @@ -19,36 +19,36 @@ packages = find: | |||
| python_requires = >=3.9 | |||
There was a problem hiding this comment.
Aber das RESPECT-Volume gibt es ja nicht normalerweise.
Vielleicht kannst du das mit dem neuen Testkram ausprobieren?
No description provided.