Navigation Menu

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

Siphon example #171

Merged
merged 6 commits into from Jan 18, 2017
Merged

Siphon example #171

merged 6 commits into from Jan 18, 2017

Conversation

ocefpaf
Copy link
Member

@ocefpaf ocefpaf commented Jan 10, 2017

@jbosch-noaa this is ready for a first review.

https://via.hypothes.is/http://nbviewer.jupyter.org/github/ocefpaf/notebooks_demos/blob/siphon/notebooks/2016-12-23-siphon-explore-thredds.ipynb

@kwilcox any idea why the line service = WebMapService(url) in cell [14] takes more than 10 minutes? That is accessing http://pdx.axiomalaska.com/ncWMS/wms

@kwilcox
Copy link
Member

kwilcox commented Jan 10, 2017

This is not pointing to the ncWMS integrated into THREDDS. We are pointing the WMS service link to a stand alone ncWMS instance that hosts every single dataset we have. The GetCaps is enormous.

We are still brainstorming how to solve this issue when we move to ncWMS2. In the meantime there isn't much we can do. Can you tell if it is response time or parsing time that takes so long?

@ocefpaf
Copy link
Member Author

ocefpaf commented Jan 11, 2017

We are pointing the WMS service link to a stand alone ncWMS instance that hosts every single dataset we have. The GetCaps is enormous.

That explains it.

Can you tell if it is response time or parsing time that takes so long?

Not sure.. I haven't checl. But that is not important for the example there. I will remove that part or hardcode a known layer just to finish that notebook.

Thanks @kwilcox!

@ocefpaf
Copy link
Member Author

ocefpaf commented Jan 18, 2017

@jbosch-noaa this should be good for another round of reviews (or merge 😉).

@jbosch-noaa jbosch-noaa merged commit 251ba3b into ioos:master Jan 18, 2017
@ocefpaf ocefpaf deleted the siphon branch January 18, 2017 19:09
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.

None yet

3 participants