Update NcISO to specify gmd:protocol for service endpoints about thredds HOT 8 CLOSED

@tomkralidis, for populating gmd:protocol we should use the identifier listed in https://github.com/OSGeo/Cat-Interop/blob/master/LinkPropertyLookupTable2.csv, correct?

from thredds.

👍

from thredds.

OK, I've submitted the PR here: Unidata/threddsIso#1
Cheers,
John

On Fri, Sep 5, 2014 at 1:14 AM, Rich Signell [email protected]
wrote:

As discussed in the pycsw issue here: geopython/pycsw#269
geopython/pycsw#269, catalog services that
rely on OWSlib like pycsw expect to have gmd:protocol specified in order to
populate the scheme in the references that get returned from a CSW request.
Without this, SOS, WMS, OpeNDAP endpoints, etc always come back with scheme
of None.

Should just need to modify the UnidataDD2MI.xsl file:

https://github.com/Unidata/thredds/blob/f902e5a3573583a9b87f1d3c7d2ff27a121289fe/tds/src/main/webapp/WEB-INF/classes/resources/xsl/nciso/UnidataDD2MI.xsl

@pacioos https://github.com/pacioos, I know you've addressed this --
can you please submit a PR to fix this?
I raised the same issue at ethanrd/threddsIso#2
ethanrd/threddsIso#2 because I wasn't sure
what the development path is on ncISO, and maybe we need to fix in both
places?

—
Reply to this email directly or view it on GitHub
#75.

from thredds.

I installed the Version 4.5.4-SNAPSHOT - 20141008.1804 snapshot of the THREDDS Data Server, and was disappointed to find out this PR to ncISO merged on Sep 8
Unidata/threddsIso@52f1bf8
has not yet been implemented in the TDS.

Do we need to submit a PR to this repo as well? I'm confused about the development chain here...

We just want to update the UnidataDD2MI.xsl file from this one:
https://github.com/Unidata/thredds/blob/4.5.4/tds/src/main/webapp/WEB-INF/classes/resources/xsl/nciso/UnidataDD2MI.xsl

to this one:
https://github.com/Unidata/threddsIso/blob/master/src/main/resources/xsl/nciso/UnidataDD2MI.xsl

from thredds.

Hi Rich,

Yes, sorry about that, I dropped the ball on that one.

We need to build a new version of threddsIso.jar so we can fold that into
the TDS build. Hopefully, all that requires is a build/deploy in
Unidata/threddsIso and an update of the threddsIso version number in the
TDS pom file.

We'll let you know when that is done.

Ethan

On Thu, Nov 6, 2014 at 6:29 AM, Rich Signell [email protected]
wrote:

I installed the Version 4.5.4-SNAPSHOT - 20141008.1804 snapshot of the
THREDDS Data Server, and was disappointed to find out this PR to ncISO
merged on Sep 8
Unidata/threddsIso@52f1bf8
Unidata/threddsIso@52f1bf8
has not yet been implemented in the TDS.

Do we need to submit a PR to this repo as well? I'm confused about the
development chain here...

We just want to update the UnidataDD2MI.xsl file from this one:

https://github.com/Unidata/thredds/blob/4.5.4/tds/src/main/webapp/WEB-INF/classes/resources/xsl/nciso/UnidataDD2MI.xsl

to this one:

https://github.com/Unidata/threddsIso/blob/master/src/main/resources/xsl/nciso/UnidataDD2MI.xsl

—
Reply to this email directly or view it on GitHub
#75 (comment).

from thredds.

@kristiansebastian, yes, you won't have to reapply the xslt after the next release of TDS!

from thredds.

Is this xsl update THREDDS version agnostic? Or only to be applied to latest version?

from thredds.

this xsl is indeed THREDDS version agnostic.
closing this because it was fixed in #114

from thredds.