Issue with reading CMEMS data - Wrong naming of functions

[joachimBach]

Hi,

I have tried utilizing the OpenDrift example CMEMS data, which utilizes the Landmask functionality. Currently if you run the function, you would have to rename the "mask.py" function to Landmask.py under the subfolder: opendrift_landmask_data in order for the model to be able to run

[knutfrode]

Hi,

I believe this should not be necessary, but is probably due to opendrift_landmask_data has been updated in the meantime.
So I would assume it should work if you update that package (and other dependencies) by:

conda env update -f environment.yml
pip install -e .

Btw, since the CMEMS example script was written, the CMEMS data have now become available directly via thredds. Thus if you put your credentials in a .netrc file, you can use this URL directly for the global current field:
https://nrt.cmems-du.eu/thredds/dodsC/global-analysis-forecast-phy-001-024-hourly-merged-uv