Implement era5 land

[geek-yang]

Support ERA5 land dataset in zampy.

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[BSchilperoort]

Good job, everything works well 🚀

I just left some comments on reorganizing the code and inheriting more.

[geek-yang]

Ok...Linter fails because of another abstract class thingy, due to protocol. Apparently the latest release of mypy is quite picky about abstract class.

[BSchilperoort]

It would be nice to keep the type hint here though:

DATASETS: dict[str, type[dataset_protocol.Dataset]] = {

Perhaps use Union[type[dataset_protocol.Dataset], type[ECMWFDataset]]? Then we get type checking in the recipe runner, otherwise the datasets there would resolve to Any.

Although I would have thought we can just provide the protocol as type and things should work: https://mypy.readthedocs.io/en/stable/protocols.html#simple-user-defined-protocols

Perhaps ECMWFDataset needs to subclass protocol as well... see: https://mypy.readthedocs.io/en/stable/protocols.html#defining-subprotocols-and-subclassing-protocols 🤔

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
3 Code Smells

98.7% Coverage
0.0% Duplication

[BSchilperoort]

Thanks for making these last changes. Let's merge! 🚀