plotly / plotly/plotly.py

Export default module in plotly-renderer.js

Ouverte
#4,789 0 commentaires 0 réactions 1 personne assignée Voir sur GitHub

Personne n'a encore pris cette issue.

feature P3
Langage dominant
Python
Étoiles
18.8k
Forks
2.8k
Merge moyen
16 h 26 min
PR mergées (30 j)
21

Description

Hi There!

We are using jupyter-plotly to support plotly plots in mystmd's web themes to enable people publishing jupyter notebooks with plotly outputs to have their plots displayed and active in their websites.

To achieve this we currently have to patch plotly-renderer.js to add a default export. See: https://github.com/jupyter-book/myst-theme/blob/3a1b70b6f2a6b827effb60891f0e693c9bf65e05/patches/jupyterlab-plotly%2B5.18.0.patch

We'd love to avoid the patch but appreciate the patch we are doing might be good for our case but break other things. If you have advice on how to change our usage, which is pretty simple (see: https://github.com/jupyter-book/myst-theme/blob/3a1b70b6f2a6b827effb60891f0e693c9bf65e05/packages/jupyter/src/plotly.ts#L40) without the need for any changes orthe patch, we'd love to go that way too.

Thanks!

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.