-
Notifications
You must be signed in to change notification settings - Fork 11
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
Unpin jupyter server #161
Unpin jupyter server #161
Conversation
platforms: | ||
- linux-64 | ||
- osx-64 | ||
- win-64 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We only build the lockfile for the main env and for these 3 platforms. We are missing ppcle, aarch64, and osx-arm64, only the last two may be worth adding here and it should be possible in the future when the migration we started in conda-forge/conda-forge-pinning-feedstock#4897 finishes.
We are hitting conda-forge/gdal-feedstock#806, let's wait a bit before we retry this PR. |
770df56
to
1b334f8
Compare
The ERDDAP interpolation notebook,
I tried 1 and 2 and failed. I guess we either ask for help or implement option 3. |
Shoot! That's a really neat notebook too. If we can identify a standard gridded dataset that can be made available to that interpolate service, we can make the data available via https://erddap.ioos.us/erddap/index.html Isn't there another dataset, on another server, that keeps failing and causes headaches for something else? Compliance checker maybe? |
The problem, for now, is the saildrone track served as tabledap in the pmel server. We probably did not get the same ban on the coastwatch one b/c the notebook errors out before reaching it.
At this moment, no. I've been changing them to more permissive servers slowly. This one is tricky though b/c I'd really like to keep that particular track. Another option, as we discussed over slack, would be a try/except clause that uses a saved CSV for the saildrone. That would ensure the notebook runs. |
And GitHub is also blocked on coastwatch :-/ Here we cannot cache the data b/c we need to access the server to demonstrate the server side interpolation.
Yep. We will have to change to a server that allows us to use it. |
Annnnnd, 15 minutes after changing the server, we are blocked again. See 2473e95 and https://github.com/ioos/ioos_code_lab/actions/runs/6171806691/job/16750780509?pr=161#step:4:173 The first commit with polarwatch was at 7:53, the second one that for us blocked happened at 8:26. This is not OK! Two access, ~30 minutes apart each :-/ |
Should we reach out to the admins? They might not know they are blocking these requests... |
I think so. Ever since I started tracking the ERDDAP servers deployments in https://github.com/ocefpaf/ERDDAP_VER_CHECK I observed some odd version downgrades every now and then and I wonder why. Maybe some of those are people who don't know they can set a better policy and are just trying to give users a better experience by downgrading. |
Self-merging this to see if I broke the site again or not with #162. |
We no longer need this workaround. The old notebook format is deprecate and we should use either nbclassic or jupyterlab. The pin was added in #134, the docs amended in #135, and this PR is the final step in the old notebook deprecation cycle.