-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Connected to CLINT data repo & added plotting #10
Conversation
@cehbrecht This change in the code requires metview (software developed at ECMWF) to be installed for plotting purposes. May I ask you to install it for shearwater? Details can be found under: https://metview.readthedocs.io/en/latest/install.html |
@cehbrecht I've tried to add the metview library requested by Michael in the environment.yaml and the requirement.txt, requirements_dev.txt as usual but I get some errors, I'll wait for you to double check it, and then I will go through the revision of the updates once the process is working. |
@PaoloBonettiPolimi @mmaiergerber tests are working now :) I switched to a "conda based build" in the tests. There was an issue with the I have updated the I tried to get the linting fixed (pep8) and the docs build (lazy load metview). Please check it the code is still doing the right thing :) The conda spec file is updated ... I will deploy it at the DKRZ VM. |
@mmaiergerber I have deployed the dev2 branch ... but it fails with the following error:
I have rolled the installation back to main branch. |
Additional Information
I was to fast and accidentally made my change in the main branch, but I immediately reverted it. Then I made the exact same change in the dev2 branch. This PR is now for you to review the changes. Thanks.