Skip to content

Fix regis URL#352

Merged
dbrakenhoff merged 5 commits intodevfrom
fix_regis_url
Jun 20, 2024
Merged

Fix regis URL#352
dbrakenhoff merged 5 commits intodevfrom
fix_regis_url

Conversation

@rubencalje
Copy link
Collaborator

@rubencalje rubencalje commented Jun 17, 2024

This PR simplifies the REGIS- and GeoTOP-urls, to the openda[-server named on https://www.dinoloket.nl/modelbestanden-aanvragen/netcdf. It also sets the version of the netcdf4 package to below 1.70, because of issue #353.

@rubencalje rubencalje marked this pull request as ready for review June 20, 2024 10:04
@rubencalje rubencalje requested a review from dbrakenhoff June 20, 2024 10:04
@bdestombe
Copy link
Collaborator

Hi Ruben,
Why did you limit the netCDF4 versions only in the CI and the creation of the documentation, and not in the main dependency list of nlmod (

"netcdf4>=1.6.3",
)?

@rubencalje
Copy link
Collaborator Author

Hi Ruben,
Why did you limit the netCDF4 versions only in the CI and the creation of the documentation, and not in the main dependency list of nlmod (

"netcdf4>=1.6.3",
)?

Because I could not reprise the error on my Windows pc, so I thought maybe it is only a linux-problem. And I did not want to restrict package versions too much.

@bdestombe
Copy link
Collaborator

Ah but what about Davíd and the few other crazy Linux users? ;)

In my case, I use nlmod as a dependency and the CI of my scripts are failing because of it. To me it would have made sense if it would have been added to the main dependecy list of nlmod.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants