Skip to content

Add jupyter-book and jupyter to setup.py#40

Merged
jdebacker merged 4 commits intojdebacker:ethfrom
rickecon:jdebacker-eth
Nov 10, 2025
Merged

Add jupyter-book and jupyter to setup.py#40
jdebacker merged 4 commits intojdebacker:ethfrom
rickecon:jdebacker-eth

Conversation

@rickecon
Copy link
Copy Markdown

@jdebacker. I think this might fix the error we are getting in the GitHub action for building the docs. There is sometimes a difference between the conda environment (ogcore-dev) and the packages that get installed from PyPI.org (pip install ogcore, from setup.py). The goal of this PR is to add the jupyter and jupyter-book packages as components of the PyPI.org ogcore package.

Review and merge this and see if this fixes the GitHub Action. I can't really test this locally on my machine.

@rickecon
Copy link
Copy Markdown
Author

@jdebacker. My most recent commit sets the version of jupyter-book<2.0.0 in both setup.py and environment.yml, and all GitHub Action tests now pass (see my PR, which I will close now). If you merge these changes into your branch, all the tests should pass now.

Comment thread setup.py Outdated
"requests",
"pip",
"pygam",
"jupyter-book<2.0.0",
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rickecon, let's remove this. We don't need JB for the ogcore package, so let's not add unnecessary dependencies.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. @jdebacker

@jdebacker jdebacker merged commit 3bd17e9 into jdebacker:eth Nov 10, 2025
7 checks passed
@rickecon rickecon deleted the jdebacker-eth branch November 10, 2025 21:08
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