Skip to content

ci(notebooks): drop ipython's test extras from the notebook requirements [11/n] - #2846

Open
bhimrazy wants to merge 1 commit into
bhimrazy/windows-bf16-skipfrom
bhimrazy/notebooks-ipython-extras
Open

bhimrazy wants to merge 1 commit into
bhimrazy/windows-bf16-skipfrom
bhimrazy/notebooks-ipython-extras

Conversation

@bhimrazy

@bhimrazy bhimrazy commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

What does this PR do?

Fixes the notebooks job, which fails at install on both GPU images.

Problem: ipython[all] includes ipython's test extras, which now pull in jupyter-ai, and through it newer PyJWT and cryptography. The image ships those two through apt, so pip can't upgrade them:

error: uninstall-no-record-file
× Cannot uninstall cryptography 41.0.7
╰─> The package's contents are unknown: no RECORD file was found for cryptography.
hint: The package was installed by debian.

Fix: keep every extra of [all] except the test ones. The notebooks don't use them.

Checked in the pt_2.8.0-dev image on an L4, following notebooks.yaml:

  • The install passes, and the image's torch and nvfuser stay as they are.
  • All 10 CI notebooks run.

Related CI:

  • notebooks.yaml on both images (pt_2.8.0-dev, pt_main-dev).

Part 11 of the breakdown of #2832.

@bhimrazy
bhimrazy added this pull request to stack #2839 September 24, 2026 09:55
@bhimrazy
bhimrazy force-pushed the bhimrazy/notebooks-ipython-extras branch from 20cbf7d to 52e82bd Compare September 24, 2026 11:10
@bhimrazy
bhimrazy removed this pull request from stack #2839 September 24, 2026 11:11
@bhimrazy
bhimrazy changed the base branch from bhimrazy/autograd-saved-for-backward-idx to bhimrazy/windows-bf16-skip September 24, 2026 11:12
@bhimrazy bhimrazy changed the title ci(notebooks): drop ipython's test extras from the notebook requirements [9/n] ci(notebooks): drop ipython's test extras from the notebook requirements [11/n] Sep 24, 2026
@bhimrazy
bhimrazy added this pull request to stack #2849 September 24, 2026 11:12
@bhimrazy
bhimrazy force-pushed the bhimrazy/notebooks-ipython-extras branch from 52e82bd to bb44ee2 Compare September 24, 2026 12:29
ipython[all] includes test_extra, which now pulls jupyter-ai -> jupyter-server-mcp
-> fastmcp -> mcp, and with it newer PyJWT and cryptography. The GPU image ships
both through apt, with no RECORD file, so pip cannot upgrade them and the
notebooks job fails at install:

  error: uninstall-no-record-file
  Cannot uninstall cryptography 41.0.7

The notebooks need none of that. Keep every other extra of [all].
@bhimrazy
bhimrazy force-pushed the bhimrazy/notebooks-ipython-extras branch from bb44ee2 to e1d5851 Compare September 24, 2026 13:44
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