Iain Russell

Results 37 comments of Iain Russell

@StephanSiemen , do you get any test failures when you run pytest? I get the same errors as are shown un the Ubuntu py3.8 tests on github.

I don't really like just removing tests, and some of those were testing something that was not being tested elsewhere, so I put some substitute tests in place. These are...

My fear is that cfgrib is storing and using the edition so that it can be used when writing an xarray to GRIB. And it *could* be important.... that said,...

@alexamici , what do you think about this PR, and is the GRIB edition important for writing xarrays back as GRIB (it seems to be)?

Hi @rrodrigojrr, my guess is that you will need local ecCodes tables in order to correctly process this data (cfgrib uses ecCodes for the actual decoding of the GRIB data)....

I think it's reasonable to remove this key because it is very specific to GRIB and not relevant to the generated NetCDF / xarray data.

Hello @jsillin , It looks to me like some of your fields are not known by ecCodes (the GRIB engine behind cfgrib). You have a variable called 'unknown' because ecCodes...

Hi @karlwx, the problem here is that you have different level types (pressure, meanSea and heightAboveGround). These are not compatible in a single xarray dataset (you could easily have the...

Thank you for this report - indeed, I think we need to make a change in the internals of cfgrib to use an 'out of range' missing value indicator. Stay...

Fixed - same resolution as for #313