-
Notifications
You must be signed in to change notification settings - Fork 500
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
Add nbconvert version check #1333
Conversation
voila/exporter.py
Outdated
@@ -20,7 +21,11 @@ | |||
from nbconvert.exporters.html import HTMLExporter | |||
from nbconvert.exporters.templateexporter import TemplateExporter | |||
from nbconvert.filters.highlight import Highlight2HTML | |||
from nbconvert.filters.markdown_mistune import IPythonRenderer, MarkdownWithMath | |||
from nbconvert.filters.markdown_mistune import ( | |||
MISTUNE_V3, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How does it work? Doesn't this fail with an import error with prior versions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indeed, my error. Updated to check for the nbconvert
version
Added a test for |
7e38d87
to
72c206c
Compare
Should this be backported? |
Benchmark reportThe execution time (in milliseconds) are grouped by test file, test type and browser. Results table
❗ Test metadata have changed--- /dev/fd/63 2023-06-23 10:50:31.397103774 +0000
+++ /dev/fd/62 2023-06-23 10:50:31.401103923 +0000
@@ -4,51 +4,49 @@
"BENCHMARK_REFERENCE": "actual"
},
"browsers": {
- "chromium": "112.0.5615.29"
+ "chromium": "94.0.4595.0"
},
"systemInformation": {
"cpu": {
- "brand": "Xeon® Platinum 8171M",
+ "brand": "Xeon® E5-2673 v3",
"cache": {
"l1d": 65536,
"l1i": 65536,
- "l2": 2097152,
- "l3": 36700160
+ "l2": 524288,
+ "l3": 31457280
},
"cores": 2,
- "efficiencyCores": 0,
"family": "6",
- "flags": "fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht syscall nx pdpe1gb rdtscp lm constant_tsc rep_good nopl xtopology cpuid pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dnowprefetch invpcid_single pti fsgsbase bmi1 hle avx2 smep bmi2 erms invpcid rtm avx512f avx512dq rdseed adx smap clflushopt avx512cd avx512bw avx512vl xsaveopt xsavec xsaves md_clear",
+ "flags": "fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht syscall nx pdpe1gb rdtscp lm constant_tsc rep_good nopl xtopology cpuid pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm abm invpcid_single pti fsgsbase bmi1 avx2 smep bmi2 erms invpcid xsaveopt md_clear",
"governor": "",
- "manufacturer": "Intel",
- "model": "85",
- "performanceCores": 2,
+ "manufacturer": "Intel®",
+ "model": "63",
"physicalCores": 2,
"processors": 1,
"revision": "",
"socket": "",
- "speed": 2.6,
+ "speed": 2.4,
"speedMax": null,
"speedMin": null,
- "stepping": "4",
- "vendor": "Intel",
+ "stepping": "2",
+ "vendor": "GenuineIntel",
"virtualization": false,
"voltage": ""
},
"mem": {
- "total": 7268679680
+ "total": 7291699200
},
"osInfo": {
"arch": "x64",
"build": "",
- "codename": "Jammy Jellyfish",
+ "codename": "Focal Fossa",
"codepage": "UTF-8",
"distro": "Ubuntu",
- "kernel": "5.15.0-1040-azure",
+ "kernel": "5.8.0-1040-azure",
"logofile": "ubuntu",
"platform": "linux",
- "release": "22.04.2 LTS",
- "serial": "b53da094229547778a51a873666147e4",
+ "release": "20.04.3 LTS",
+ "serial": "cfc067bfcb844f35865e279a1b0e66c5",
"servicepack": "",
"uefi": false
} |
It looks like GitHub is running out of osx runner |
Yes, only to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Let's not wait for the OSX build
@meeseeksdev please backport to 0.4.x |
Owee, I'm MrMeeseeks, Look at me. There seem to be a conflict, please backport manually. Here are approximate instructions:
And apply the correct labels and milestones. Congratulations — you did some good work! Hopefully your backport PR will be tested by the continuous integration and merged soon! Remember to remove the If these instructions are inaccurate, feel free to suggest an improvement. |
* Add mistune version check * check nbconvert version * Update test (cherry picked from commit f416ab5)
References
Due to the non-backward compatible change in
nbconvert
7.6.0 on markdown rendering,VoilaMarkdownRenderer
class needs to be aware of thenbconvert
version.Code changes
User-facing changes
Backwards-incompatible changes