Commit fc7f702
authored
feature: experimental gribjump source (#689)
* wip: first rough draft of a GribJumpSource
* wip: experimental tests for easier development
* format changes using pre-commit hooks and fix small bug
* add prototype for the GribJumpSource based on SimpleFieldList
* tests: add a few more simple tests and add NO_GRIBJUMP flag for pytest
* tidy: small cleanup, improve variable naming and fix type hints
* use original type in request dictionaries to make .sel more intuitive
* assign grid index to each value in to_xarray
* tidy: add some more error handling and improve tests
* refactor: introduce ExtractionRequest wrapper that combines pygribjump.ExtractionRequest and original fdb request dict
* feat(test): modify (now failing) test to expect latitude and longitude information
* feat: wip: allow reference lat/lons to be loaded from an fdb reference field
* refactor: move hardcoded test fixtures into pytest fixtures
* test: add failing test showing bug with geography for gridded extracts
* docs: add notebook draft with example usage of gribjump source
* tidy: move validation that extract request share the same ranges
* docs: add documentation for gribjump source
* docs: small fixes of markdown syntax
* feat: wip experiment to verify gridspec of reference field
* fix: force flattened array in xarray dataset creation
* refactor: tidy up the metadata enrichment a bit
* refactor: create ExtractionRequestCollection
* refactor: use FDBRetriever to load reference metadata
* docs: add example for masks and indices to notebook
* feat: enforce that masks are 1D boolean arrays
* refactor: simplify by condensing request splitting utilities into one function
* feat: remove verifiation functionality for now, to be added later
* tidy: small renamings and docstrings
* tidy: comments
* fix: type hint and name
* feat: convert masks to ranges once for significant speedups
* test: add another test for mask_to_ranges
* tidy: small comment/docstring changes
* tidy: make warning about missing validation in docs more explicit
* docs: improve wording of warning
* fix: allow fdb and gribjump to be configured via FDB5_CONFIG and GRIBJUMP_HOME
* chore: update docstring
* feat: pass log context to gribjump
* refactor: simplify gribjump log context
* test: add t_gribjump.grib test data with expver xxxx
* add pygribjump as an optional dependency
* docs: clarify gribjump install instructions and dependency handling
* docs: move warning before parameters section
* docs: clarify parameter description and types
* add pyfdb as a gribjump group dependency and update docs
* last docs and typo fixes
* docs: change notebook to also set FDB_HOME
* docs: reference gribjump example notebook in missing locations1 parent a312ba5 commit fc7f702
File tree
10 files changed
+2095
-12
lines changed- docs
- examples
- guide
- src/earthkit/data
- sources
- tests
- data
- documentation
- sources
10 files changed
+2095
-12
lines changedLarge diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
| 33 | + | |
33 | 34 | | |
34 | 35 | | |
35 | 36 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
66 | 66 | | |
67 | 67 | | |
68 | 68 | | |
| 69 | + | |
| 70 | + | |
69 | 71 | | |
70 | 72 | | |
71 | 73 | | |
| |||
1231 | 1233 | | |
1232 | 1234 | | |
1233 | 1235 | | |
| 1236 | + | |
| 1237 | + | |
| 1238 | + | |
| 1239 | + | |
| 1240 | + | |
| 1241 | + | |
| 1242 | + | |
| 1243 | + | |
| 1244 | + | |
| 1245 | + | |
| 1246 | + | |
| 1247 | + | |
| 1248 | + | |
| 1249 | + | |
| 1250 | + | |
| 1251 | + | |
| 1252 | + | |
| 1253 | + | |
| 1254 | + | |
| 1255 | + | |
| 1256 | + | |
| 1257 | + | |
| 1258 | + | |
| 1259 | + | |
| 1260 | + | |
| 1261 | + | |
| 1262 | + | |
| 1263 | + | |
| 1264 | + | |
| 1265 | + | |
| 1266 | + | |
| 1267 | + | |
| 1268 | + | |
| 1269 | + | |
| 1270 | + | |
| 1271 | + | |
| 1272 | + | |
| 1273 | + | |
| 1274 | + | |
| 1275 | + | |
| 1276 | + | |
| 1277 | + | |
| 1278 | + | |
| 1279 | + | |
| 1280 | + | |
| 1281 | + | |
| 1282 | + | |
| 1283 | + | |
| 1284 | + | |
| 1285 | + | |
| 1286 | + | |
| 1287 | + | |
| 1288 | + | |
| 1289 | + | |
| 1290 | + | |
| 1291 | + | |
| 1292 | + | |
| 1293 | + | |
| 1294 | + | |
| 1295 | + | |
| 1296 | + | |
| 1297 | + | |
| 1298 | + | |
| 1299 | + | |
| 1300 | + | |
| 1301 | + | |
| 1302 | + | |
| 1303 | + | |
| 1304 | + | |
| 1305 | + | |
| 1306 | + | |
| 1307 | + | |
| 1308 | + | |
| 1309 | + | |
| 1310 | + | |
| 1311 | + | |
| 1312 | + | |
| 1313 | + | |
| 1314 | + | |
1234 | 1315 | | |
1235 | 1316 | | |
1236 | 1317 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
51 | 51 | | |
52 | 52 | | |
53 | 53 | | |
| 54 | + | |
54 | 55 | | |
55 | 56 | | |
56 | 57 | | |
| |||
85 | 86 | | |
86 | 87 | | |
87 | 88 | | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
50 | | - | |
| 50 | + | |
51 | 51 | | |
52 | 52 | | |
53 | 53 | | |
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
| 73 | + | |
73 | 74 | | |
74 | 75 | | |
75 | 76 | | |
| |||
0 commit comments