You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Ncml in featureCollections is not used or tested. The issue is probably in the DatasetManager::openNetcdfFile and DatasetManager::openCoverageDataset. We should add integration tests for both cases, e.g. through WMS/ opendap and opening as a coverage through NCSS. Check that both metadata and data changes work with ncml (e.g. renaming a variable and applying an enhancement such as the Standardizer).
NcML is no longer used to define the collection, but it may still be used to modify the feature collection dataset for FMRC or Point (not GRIB).
I have manually tested featureType=FMRC, and that does seem to work with NcML. And the above says it should not work with GRIB1/GRIB2. Ncml does not seem to work with featureType=Point or featureType=station, so either that needs to be fixed or the documentation should be updated.
The text was updated successfully, but these errors were encountered:
tdrwenski
changed the title
featureCollections don't use ncml
Point/Station featureCollections don't use ncml
Jul 28, 2023
Ncml in featureCollections is not used or tested. The issue is probably in the
DatasetManager::openNetcdfFile
andDatasetManager::openCoverageDataset
. We should add integration tests for both cases, e.g. through WMS/ opendap and opening as a coverage through NCSS. Check that both metadata and data changes work with ncml (e.g. renaming a variable and applying an enhancement such as the Standardizer).See the featureCollection documentation:
I have manually tested
featureType=FMRC
, and that does seem to work with NcML. And the above says it should not work withGRIB1
/GRIB2
. Ncml does not seem to work withfeatureType=Point
orfeatureType=station
, so either that needs to be fixed or the documentation should be updated.The text was updated successfully, but these errors were encountered: