CliMA / ClimaOcean.jl

🌎 Framework for realistic regional-to-global ocean simulations, and coupled ocean + sea-ice simulations based on Oceananigans and ClimaSeaIce. Basis for the ocean and sea-ice component of CliMA's Earth system model.
https://clima.github.io/ClimaOceanDocumentation/stable/
Other
34 stars 9 forks source link

Not all `wget`s have `http-passwd` #272

Open Sbozzolo opened 2 days ago

Sbozzolo commented 2 days ago

My guess is that it was removed with wget2, released in 2021.

glwagner commented 2 days ago

@simone-silvestri is there a way we can use Base.downloads? @Sbozzolo thinks so.

simone-silvestri commented 2 days ago

We should write a .netrc file like they do in here: https://github.com/evetion/SpaceLiDAR.jl/blob/688105ed50c71e9d8d02c52c7a83801da1728611/src/utils.jl#L122

glwagner commented 2 days ago

I think there is also a way to create a downloader than can do this right? I'm looking at the documentation:

https://github.com/JuliaLang/Downloads.jl

Here's what the ECCO website says to do:

image

We aren't doing what they recommend though:

https://github.com/CliMA/ClimaOcean.jl/blob/8057d1a18ac16ad3ee3e4fc416868062e4e0d2de/src/DataWrangling/ECCO/ECCO_metadata.jl#L237

Still probably better to avoid cmd.

glwagner commented 2 days ago

Why do we use http-user instead of user?

Sbozzolo commented 2 days ago

Yes, I think you should be able to create a custom downloader that posts the user and the password as part of the request (but I have never done anything like this).