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

Do not plot matplotlib vector graphics by default #221

Closed
4 tasks done
redeboer opened this issue Nov 27, 2023 · 0 comments · Fixed by #223, ComPWA/qrules#237, ComPWA/ampform#368, ComPWA/tensorwaves#507 or ComPWA/compwa.github.io#221
Assignees
Labels
🖱️ DX Improvements to the Developer Experience

Comments

@redeboer
Copy link
Member

redeboer commented Nov 27, 2023

Suggestion

The set-nb-cells sub-hook currently adds the statement %config InlineBackend.figure_formats = ['svg'] to the top of each notebook. This is useful if there are matplotlib plots where we want vector graphics. In many cases, though, if you plot something like a pcolormesh with high resolution and are not aware of this config statement, the notebook crashes, because these figures are not suited for vector graphics.

Proposed solution

Remove the statement, but check which notebooks of repositories that use the set-nb-cells hook do require vector graphics.

Repositories

  1. 1 of 1
    🖱️ DX
    redeboer
  2. 1 of 1
    🖱️ DX
    redeboer
  3. 1 of 1
    🖱️ DX
    redeboer
  4. 1 of 1
    🖱️ DX
    redeboer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment