-
Notifications
You must be signed in to change notification settings - Fork 179
UGRID with TDS and edal-java/ncwms #526
Comments
Ok, so there is a bit of a hangup here, in the the new version godiva3 is causing an error with the TDS, in that it is trying to call:
which currently returns a 404. Note that wms seems to be working just fine - it's a godiva3 thing that we have not implemented in the TDS. I need to look into what exactly godiva3 is looking for here and make sure we return that. @guygriffiths - any pointers to documentation as to what exactly godiva3 wants from the getconfig request? Thanks in advance! |
That shouldn't be a problem. It exists so that Godiva3 can get settings about the map size, data proxy, and the location of the documentation from a server. It was written for a specific project - we don't actually use it in the default ncWMS2 implementation, but saw no reason to remove it. Best thing is to ignore it. |
So we figured we would help out here and get TDS 5.0 installed locally where we have a lot of UGRID data (and the provider is very keen to get WMS services going). We installed TDS 5.0 using the docker container from https://hub.docker.com/r/axiom/docker-thredds/ at http://www.smast.umassd.edu/thredds/catalog/catalog.html and although OPeNDAP service and HTTPS services are working okay, the WMS service is not working. For example, on this UGRID dataset: If we click the WMS link (http://www.smast.umassd.edu/thredds/wms/FVCOM/NECOFS/Forecasts/NECOFS_GOM3_FORECAST.nc?service=WMS&version=1.3.0&request=GetCapabilities) we get:
So it would seem like there is some sort of path issue. We can see that WMS |
BTW, if you are wondering which version of TDS 5.0 we used, we pulled the latest artifact from your nexus repo. |
@lesserwhirls ping! |
Hi @rsignell-usgs - the snapshot you are using is newer than the one on thredds-test.unidata.ucar.edu. I'm building the latest locally and going to test the wms on my machine, and if everything looks good, I'll publish a new set of 5.0 snapshots. Godiva3 still won't function, but we are digging into that (not sure why the WMS isn't working in the snapshot you are using though). |
@lesserwhirls , awesome. While Godiva3 would be nice, we really care most about the WMS and ISO services. If those are working we will be able to harvest ISO metadata from UGRID complaint data on TDS5.0 servers and access the ncWMS2 WMS services in our portals like |
Ok, I'm currently publishing a new snapshot which should work. I will be out all next week, but I can try to get threddsIso integrated into 5.0 during the evenings. I have a version that builds with the new 5.0 stuff, but it's not yet tied into the TDS. |
@kwilcox, can you rebuild the Docker container for 5.0 with this artifact? |
Moved this artifact discussion to #495 (comment) |
@rsignell-usgs - have you tried the wms service using that snapshot for a regular gridded dataset? I think I need to tweak a few things on the TDS side to recognize the possibility for ugrid to work with wms before it will actually pass the dataset to the wms server (this hit me about half-way through Kansas on my way back from Indiana...something about the plains I suppose). |
Yes, when I hit the WMS Capabilities on this regular structured-grid FMRC endpoint: I get back:
|
Is there something different required in the catalog for FMRC in 5.0? |
The 5.0 catalogs used on thredds-jumbo (which is actually thredds-dev) can be found here: https://github.com/Unidata/TdsConfig/tree/master/threddsDev Not sure why wms isn't working with your gridded dataset...only thing I can think of is something in FMRC, but I'm not positive. |
Ah, looks like FMRC stuff did change - see: https://www.unidata.ucar.edu/software/thredds/v5.0/tds/reference/collections/FmrcCollection.html |
@lesserwhirls, that didn't change in 5.0 though -- it changed in 4.2. We have been using I just checked and none of our WMS endpoints are working -- even regular bathymetry grids like when I click on the WMS getcaps endpoint are reporting back:
Does that give any clues? |
This is so odd. I cannot even grab the bathymetry grid for testing locally using the HTTPServer (I get a 404). Would you mind if I took a look at your threddsConfig.xml, the catalog used to serve up the bathymetry grid, as well as your top level catalog.xml file? |
Okay, here are the files. https://drive.google.com/open?id=0BzAHlPEEP_ujM3p2MEl5TmUwajg |
I've moved the issue of simple grids not being accessible to the new ncWMS to #618. We'll keep this one open for UGRID specific discussions once we get the simple grids to work. |
Unidata is packaging TDS 5.0 for Docker: https://github.com/Unidata/thredds-docker Is this working? |
Please use the docker container from the Unidata org! Everything from the axiom container was merged into it. I really need to take down the Axiom one. |
Does FMRC stuff work on UGRID datasets? I saw old discussions around http://www.unidata.ucar.edu/mailing_lists/archives/thredds/2013/msg00503.html but I can't seem to make it work. |
@drf5n, AFAIK, FMRC currently requires This is especially frustrating because the logic in constructing the FMRC doesn't have anything to do with the horizontal grid system. So if NetCDF-Java recognined Is that right @lesserwhirls ? |
That's a really good question @rsignell-usgs! I think netCDF-Java still needs to be able to recognize the data as "gridded data", which means that it understands the horizontal coordinate system (even though FMRC might not to anything with that info). |
@rsignell-usgs, based on your old email, I seem to be able to make my ugrid data regognizable as dataType='grid' with a call to NCO's
|
@rsignell-usgs -- See #778 (comment) for hints at an FMRC-ish workaround with NCML's I wonder what in NetCDF-Java recognizes UGRID files with the added degenerate dimension as dataType=grid? If the coordinate's rank is >1 it passes? |
Thanks @drf5n! @lesserwhirls, to me this indicates how easy it should be to allow a UGRID to be an FMRC! 😸 |
I agree, these results look very promising! What I'd like to do is hammer out |
Get a UGRID example from @rsignell-usgs and see how it works with TDS 5 and the new edal-java/ncwms plugin.
The text was updated successfully, but these errors were encountered: