-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Does threddsIso handle the 'source' CF attribute? #34
Comments
As of ACDD 1.1 there was no mapping or use of the global http://wiki.esipfed.org/index.php/Attribute_Convention_for_Data_Discovery_1-1
|
Hey @lukecampbell. I read down to this comment in issue #7 noticed that is is included in the new mapping spreadsheet. I don't know how long Hopefully, it will be included in the upcoming release! (this will help us tremendously at IOOS to identify model datasets by some automated fashion for potential inclusion in EDS via the Catalog, at least, if CKAN indexes where it ends up in the ISO XML for search/filter, a separate issue potentially). See: ioos/catalog#40 |
Just to xref. another issue here: this would really help with ioos/catalog#40 |
Tagging @noaaroland - do the transforms in https://github.com/NOAA-PMEL/uafnciso handle the source attribute? |
The CDN versions of the transforms I use in the code are: |
@noaaroland Thanks for updating the NOAA-PMEL/uafnciso repo with fixes! Since this is a fork of the original ncISO, I presume it won't be included in any new TDS releases, correct? We can point our data providers to use the uafnciso release JAR files as an alternative to generate ISO files from TDS. If, that is, we are able to leverage that for filtering capability based on the values populated in 'source' - one of the original goals of ioos/catalog#40. Not sure if this should be closed or left open until this functionality is ported here if that's desired. Feel free to close if not. |
I think we are still in a place where there have been changes made to the transforms in this repository, and there have been changes made to the transforms in the NOAA-PMEL repository, and they have diverged enough to where I am unable to make a call on what stays and what goes (plus, I'm not exactly proficient with xslt stuff, so you probably don't want me doing it anyways). It'd be great if we could settle around a canonical set, and I'd be in favor of adopting them here (not my call, however). |
I agree with consolidating the changes in the Unidata threddisIso repo, I think that was the intent behind merging the client code into that repository as well. Not sure who is best positioned to review and merge the XSLT at this point? @amilan17 @noaaroland |
We have also made some changes to XSLT to address some of the issues such as: |
See: http://cfconventions.org/cf-conventions/v1.6.0/cf-conventions.html#description-of-file-contents.
I did a search in the current version of UnidataDD2MI.xsl and couldn't find it called out in the variables section at the top.
Since this is mentioned as a critical file description variable in CF, shouldn't it be parsed and output in the ISO XML somewhere?
The text was updated successfully, but these errors were encountered: