search

Reading-eScience-Centre / edal-java

Environmental Data Abstraction Layer libraries
Other
39 stars 30 forks source link

ncWMS2 GetCapabilities Issue After Adding Dataset #29

Closed vansant closed 8 years ago

vansant commented 9 years ago

Running into a WMS GetCapabilities issue after adding a dataset in ncWMS2. ncWMS2 is running on Apache Tomcat using the .war at this link https://github.com/Reading-eScience-Centre/edal-java/releases/download/edal-1.0.2/ncWMS2.war

The GetCapabilities links for 1.1.1 and 1.3.0 both work until a dataset gets added via the ncWMS admin. After the configuration is saved the dataset loads into GODIVA3 and also as PNG and KMZ files. Although, the GetCapabilities links are returning invalid XML (incomplete tags) when a dataset is available.

Youtube video demo of issue: https://www.youtube.com/watch?v=LRbiOrogEBQ

Invalid XML document shown in video: http://pastebin.com/QbavzNBW

Thoughts?

-VanSant

vansant commented 9 years ago

Also, this issue is present when using the standalone version of ncWMS2.

-VanSant

guygriffiths commented 9 years ago

Can you provide us with the dataset you are using? I've tried to replicate this with all of our test datasets but have been unable to get the same result.

I can however replicate the error by throwing a runtime exception when trying to get the GeographicBoundingBox of the HorizontalDomain. I suspect that this is what's happening, but I need some data which does this in a non-artificial way to find out what the root cause is.

vansant commented 9 years ago

Thank you for your quick response/info. Yes, two links/options for acquiring the dataset are provided below.

Here is a direct link to download the dataset which is 2.3 GB: http://thredds.northwestknowledge.net:8080/thredds/fileServer/NWCSC_INTEGRATED_SCENARIOS_ALL_CLIMATE/macav2livneh/bcc-csm1-1-m/macav2livneh_pr_bcc-csm1-1-m_r1i1p1_historical_1950_1969_CONUS_daily.nc

Here is a link to a NetCDF subset tool (THREDDS) where you can slice out a year or two to make the file size much smaller for downloading: http://thredds.northwestknowledge.net:8080/thredds/ncss/grid/NWCSC_INTEGRATED_SCENARIOS_ALL_CLIMATE/macav2livneh/bcc-csm1-1-m/macav2livneh_pr_bcc-csm1-1-m_r1i1p1_historical_1950_1969_CONUS_daily.nc/dataset.html

Best,

-VanSant

guygriffiths commented 9 years ago

Great, thanks. I've tracked down the issue and it's fixed in the develop branch. It was occurring when the bounding box of the data had longitudes specified in the range [0:360]. We'll do a bug-fix release soon to include this.

vansant commented 9 years ago

Excellent news!

Thanks again,

-VanSant

vansant commented 8 years ago

Hello,

Any news on when the bug-fix release is planned for this?

Best,

VanSant

guygriffiths commented 8 years ago

I'm planning on doing a bugfix release Friday afernoon

guygriffiths commented 8 years ago

I got around to this earlier than expected - it's fixed in the latest release