0.5.27

[peterdudfield]

openclimatefix/pv-site-production#91

Pv-site-production didnt seem to work tieh 0.5.27, but was fine with 0.5.25. Im not sure what the issue is to be honest

[peterdudfield]

Nor does pvnet

[peterdudfield]

I this could be related to the variables not be renamed proberly

[peterdudfield]

Could be good idea to pin requirements to make sure this doesnt happy

[devsjc]

I'll see if I can pin down whether its something that has changed in the cfgrib or eccodes library, as they are what decode the grib messages - I can't think of any changes in the repository itself that would lead to this otherwise, that test file and the reading code hasn't been touched in a long time! If so, as you say, an explicit version pin should resolve it.

[peterdudfield]

Was the idea to pin the requirments, similar to your local to check it works ok?

[devsjc]

It was, see #186

[devsjc]

Fixed with #186

[peterdudfield]

Sorry im still seeing issues with 0.5.27

[peterdudfield]

we should put this test back in -

nwp-consumer/src/nwp_consumer/internal/inputs/metoffice/test_client.py

Line 31 in 51cefba

@unittest.skip("Broken on github ci")

[peterdudfield]

Im just trying to run the docker file made on main to see if we have the same issue

[peterdudfield]

started a draft PR - #187

[peterdudfield]

Annoying thing is the test works fine locally, but doesnt work in the CI and docker file e.t.c.

i might try builind the nwp docker file locally, and run the test in that

[peterdudfield]

could be related to ecmwf/cfgrib#385?

[peterdudfield]

Looks like it is solved in 0.5.28