Skip to content
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

Improve NCSS docs and upgrade guide #545

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

lesserwhirls
Copy link
Collaborator

Call out changes to NCSS service elements in 5.x (i.e. GRID and POINT have different service elements). Addresses #544.

Call out changes to NCSS service elements in 5.x (i.e. GRID and POINT have different service elements). Addresses Unidata#544.
@lesserwhirls lesserwhirls marked this pull request as ready for review November 20, 2024 16:02
@tlvu
Copy link

tlvu commented Nov 20, 2024

Wow, so many places !

Should there be an example with both Grid and Point enabled simultaneously at the same time for mixed datasets? Are there any bad side-effect if both are enabled at the same time?

tlvu added a commit to bird-house/birdhouse-deploy that referenced this pull request Nov 20, 2024
@lesserwhirls
Copy link
Collaborator Author

Yep, a lot of place for sure (and several places already had the correct info). A few years back we started to break out the what we called the TDS user guide into four pieces - a server admin guide, a server admin quick start guide, an end users guide, and a tds for developers guide. There is currently a ton of duplication, but I will keep the files in sync for the time being until we can get things cleanly separated.

At any rate, mixed Point and Grid FeatureType datasets (see these netCDF-Java docs for more info) are not supported in NCSS. NCSS still supports point requests against gridded data (e.g. extract time series data from a grid point), but that all falls under the /thredds/ncss/grid service for "gridded" data.

@lesserwhirls lesserwhirls merged commit 67ef459 into Unidata:main Nov 21, 2024
5 checks passed
@lesserwhirls lesserwhirls deleted the ncss_docs branch November 21, 2024 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants