Skip to content

Switch to pixi#190

Open
jl-wynen wants to merge 19 commits into
mainfrom
pixi
Open

Switch to pixi#190
jl-wynen wants to merge 19 commits into
mainfrom
pixi

Conversation

@jl-wynen

Copy link
Copy Markdown
Collaborator

Fixes #189

Testing the setup first. Still need to update the README.

@jl-wynen jl-wynen marked this pull request as ready for review June 23, 2026 10:38
@jl-wynen

Copy link
Copy Markdown
Collaborator Author

CI is failing because a linkcheck of mantidproject.org and that website is down.

@@ -22,9 +22,6 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
- uses: actions/checkout@v4
- uses: actions/checkout@v7

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I'll do that in the same PR as Python updates.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

See #198

@@ -42,25 +39,15 @@ jobs:
steps:
- uses: actions/checkout@v4

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
- uses: actions/checkout@v4
- uses: actions/checkout@v7

There might also be other actions that need updating?

Comment thread pixi.toml
platforms = ["linux-64", "osx-64", "osx-arm64", "win-64"]

[dependencies]
python = "3.11.*"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
python = "3.11.*"
python = "3.12.*"

Wouldn't this be the opportunity to bump python (maybe even 3.13?)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I'll update dependencies in a separate PR (in progress). This one was meant to just migrate to pixi.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

(maybe even 3.13?)

I'd only go to 3.12 because visa uses it.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

See #198

Comment thread pixi.toml
Comment on lines +32 to +34
scipp = "=25.5"
scippneutron = "=25.7"
scippnexus = "=25.6"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we update to v26 packages?

Comment thread pixi.toml
cwd = "./"

[tasks.linkcheck]
cmd = "jupyter-book build -W --keep-going --builder linkcheck ."

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does jupyter-book also support the -D nbsphinx_execute=never as we did here to avoid running notebooks when we are just checking links?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I don't think so:

Usage: jupyter-book build [OPTIONS] PATH_SOURCE

  Convert your book's or page's content to HTML or a PDF.

Options:
  --path-output TEXT              Path to the output artifacts
  --config TEXT                   Path to the YAML configuration file
                                  (default: PATH_SOURCE/_config.yml)
  --toc TEXT                      Path to the Table of Contents YAML file
                                  (default: PATH_SOURCE/_toc.yml)
  -W, --warningiserror            Error on warnings.
  -n, --nitpick                   Run in nit-picky mode, to generates warnings
                                  for all missing references.
  --keep-going                    With -W, do not stop the build on the first
                                  warning, instead error on build completion
  --all                           Re-build all pages. The default is to only
                                  re-build pages that are new/changed since
                                  the last run.
  --builder [html|dirhtml|singlehtml|pdfhtml|latex|pdflatex|linkcheck|custom]
                                  Which builder to use.
  --custom-builder TEXT           Specify alternative builder provided by
                                  Sphinx, including text and epub. This can
                                  only be used with --builder=custom. Valid
                                  options listed at https://www.sphinx-
                                  doc.org/en/master/man/sphinx-build.html
  -v, --verbose                   increase verbosity (can be repeated)
  -q, --quiet                     -q means no sphinx status, -qq also turns
                                  off warnings
  --individualpages               [pdflatex] Enable build of PDF files for
                                  each individual page
  -h, --help                      Show this message and exit.

In CI that should be fine because the notebooks are cached from the build. You can see in the logs that it doesn't re-execute the notebooks.

If you wanted to force this, I think you would need to modify _config.yml. Or maybe it is possible via the CLI of jupyter-book v2. (Need to migrate to that as well)

@jl-wynen jl-wynen mentioned this pull request Jun 23, 2026
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.

Maybe switch to pixi

3 participants