search

ARPA-SIMC / arkimaps

generazione mappe meteorologiche da modelli previsionali
GNU General Public License v2.0
0 stars 1 forks source link

lint: "cannot be parsed" eccodes noise #172

Closed brancomat closed 2 months ago

brancomat commented 2 months ago

While the recipe validation is really useful (thanks) the lint option produces a lot of noise for each and every eccodes match, e.g.:

WARNING arkimaps.lint tp_inputs.yaml, input tp: eccodes match 'centre != 98 and shortName is "tp" and editionNumber == 1' cannot be parsed: []

Is there any way to circument these warnings? Alternatively, would it be possible to replace them with a single generic warning such as

WARNING arkimaps.limt eccodes expressions won't be parsed/validated

?

spanezz commented 2 months ago

grib_filter changed error output between the distro I developed this on, and the distro you tested this on: I now changed the test using a regexp that should match both.

There's no way to ask grib_filter to test if an expression is correct, so I had to hack it a bit