Skip to content

fix: some small changes for glambie submission - #52

Merged
tsutterley merged 1 commit into
mainfrom
glambie
Jul 13, 2026
Merged

fix: some small changes for glambie submission#52
tsutterley merged 1 commit into
mainfrom
glambie

Conversation

@tsutterley

Copy link
Copy Markdown
Owner

docs: convert NASA Earthdata to notebook
feat: add new pixi task
docs: compile some of the documentary notebooks

docs: convert NASA Earthdata to notebook
feat: add new pixi task
docs: compile some documenting notebooks
@tsutterley

Copy link
Copy Markdown
Owner Author

@copilot review this pull request ignoring the pixi.lock file. Only review the changes and make suggestions. Do not add any new commits.

Copilot AI commented Jul 13, 2026

Copy link
Copy Markdown

@copilot review this pull request ignoring the pixi.lock file. Only review the changes and make suggestions. Do not add any new commits.

One substantive issue stood out:

  • doc/source/conf.py:65-79 now enables notebook execution for RTD/local docs builds, but the newly added doc/source/getting_started/Install.ipynb and doc/source/user_guide/NASA-Earthdata.ipynb contain non-hermetic cells (! pixi task list and a live IS2view.utilities.cmr(...) network query). Because those notebooks have no stored outputs, myst-nb will try to execute them during make html, which can make docs builds fail when pixi is unavailable or external access/authentication is not available.

Suggested fix: keep those notebooks non-executed during Sphinx builds, or mark the environment/network-dependent cells to be skipped.

@tsutterley
tsutterley merged commit 8c53d4a into main Jul 13, 2026
5 checks passed
@tsutterley
tsutterley deleted the glambie branch July 13, 2026 22:13
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