SDSlite icon indicating copy to clipboard operation
SDSlite copied to clipboard

Enable read-only NetCDF DAP uri:s

Open juselius opened this issue 3 years ago • 4 comments

Prevent (local) file existence checks for NetCDF uri:s which start with http. Instead pass directly to nc_open_chunked(), to enable read-only DAP/THREDSS support.

juselius avatar Feb 25 '22 19:02 juselius

Sorry for modifying white space. My editor removes them, and was too tired to put them back.

juselius avatar Feb 25 '22 19:02 juselius

@juselius, thank you for your interest in contributing to SDSlite. The solution you suggest will prevent treating properly files named http.nc and alike. Can you pls create an issue that documents current behaviour with DAP urls?

vassilyl avatar Feb 27 '22 13:02 vassilyl

Hi @vassilyl, I'll provide a better solution soon, so that it's done properly. The problem is that I'm quite rusty in C#, since I work mostly in F# on .NET. I already tried to implement a DapUrl property in the URI, but there we so many levels of inheritance and so little time, that I just hacked the FileName property instead to get a working solution.

juselius avatar Feb 27 '22 19:02 juselius

@vassilyl is it better now? I have added a Url property for DAP access to the Uri, and refactored things a bit to avoid duplication.

juselius avatar Feb 28 '22 14:02 juselius