Skip to content
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

[ci] Specify nbconvert version in requirements.txt #966

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

vepadulano
Copy link
Member

Recent failures of roottest-python-JupyROOT-importROOT_notebook (specifically on Fedora 37) would report the message

[NbConvertApp] Converting notebook importROOT.ipynb to notebook
[NbConvertApp] Writing 988 bytes to importROOT_out.nbconvert.ipynb
...
    File "/tmp/workspace/src/roottest/python/JupyROOT/nbdiff.py", line 68, in getFilteredLines
      filteredLines = list(filter(customLineJunkFilter, open(fileName).readlines()))
                                                        ^^^^^^^^^^^^^^
  FileNotFoundError: [Errno 2] No such file or directory: 'importROOT_out.ipynb'

This is due to a bug in nbconvert version 7.3, as described in jupyter/nbconvert#1970. Other versions of the package do not suffer from the bug, so we update the requirements.txt file accordingly.

Recent failures of roottest-python-JupyROOT-importROOT_notebook
(specifically on Fedora 37) would report the message

```
[NbConvertApp] Converting notebook importROOT.ipynb to notebook
[NbConvertApp] Writing 988 bytes to importROOT_out.nbconvert.ipynb
...
    File "/tmp/workspace/src/roottest/python/JupyROOT/nbdiff.py", line 68, in getFilteredLines
      filteredLines = list(filter(customLineJunkFilter, open(fileName).readlines()))
                                                        ^^^^^^^^^^^^^^
  FileNotFoundError: [Errno 2] No such file or directory: 'importROOT_out.ipynb'
```

This is due to a bug in nbconvert version 7.3, as described in
jupyter/nbconvert#1970. Other versions of the
package do not suffer from the bug, so we update the requirements.txt
file accordingly.
@vepadulano vepadulano self-assigned this May 18, 2023
@phsft-bot
Copy link

Starting build on ROOT-debian10-i386/soversion, ROOT-performance-centos8-multicore/cxx17, ROOT-ubuntu18.04/nortcxxmod, ROOT-ubuntu2004/python3, mac12/noimt, mac11/cxx14, windows10/cxx14
How to customize builds

@vepadulano vepadulano merged commit 0bdebb1 into root-project:master Jul 19, 2023
@hahnjo
Copy link
Member

hahnjo commented Jul 20, 2023

It turns out this doesn't actually work and break CI image generation; fix in #984

edit: Vincenzo was faster, #983

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.

4 participants