plotly core (a community vote for a simple wrapper to plotly.js)
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
- Type d'issue
- Fonctionnalité
- Clarté
- À clarifier
- Activité
- Calme
- Stack technique
- javascript, r
- Domaine
- data-visualization
Piste de recherche
Commencez par lire la séparation proposée entre un wrapper simple de plotly.js et le package compatible avec ggplotly, puis comparez l’exemple fst/fstcore lié dans l’issue. Examinez la structure actuelle du package plotly.R ainsi que les responsabilités de plot_ly(), ggplotly et regression-test. Le travail est terminé lorsque les maintainers se sont mis d’accord sur des limites de package et des objectifs d’implémentation concrets.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Hello, long time user of plotly.R here. I thought this issue could start the discussion to split up the package into a core and a full ggplotly supported package.
As many have noted, I also started using plotly from the ggplot2 interface of ggplotly,
but as times are changing, and I am getting to understand the drawbacks of wrapping things through ggplot more,
we are moving more and more away from it.
In our RiboCrypt app, we have for optimizations reasons removed all ggplotly calls and transformed to native plot_ly() calls. We experiment with optimizations like pre-building plotly template objects, to speed up "time to user see something useful".
And since this is an app for complicated genomics, R/bioconductor is still a far better fit than python.
A problem has been the last few year that new plotly.R versions break stuff, so I have to keep version set to a version I know works. Problem is that new version needs to pass not just plotly.js tests, but also ggplotly tests for regression, so it is currently much more complicated than it needs to be.
I vote for a clean split of the package, for existing popular R package doing this, see for example the fst package, which has fst and fstcore
plotlycore: Simple wrapper to plotly.js
plotly: Uses plotlycore + ggplotly interface.
This way I think it is more safe especially for people using it for actual production apps and not just scripting.
I thought we could discuss here what people have noticed lately, what the relevant goals of current maintainers are, what resources exist to do this etc.
- Langage dominant
- R
- Étoiles
- 2.7k
- Forks
- 641
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de plotly/plotly.R
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
-
Data gets dropped in add_trace and consequently points have the wrong color if NA is present Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
-
save_image Error Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
Toutes les issues de plotly/plotly.R
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
easystats/performance#950 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
briandconnelly/airnow#9 ·
-
Copy cohorts to keep old cohorts Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
OHDSI/CohortConstructor#774 ·
-
Ensure HTTPS is enabled Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100