how about removing function last_plot?
Personne n'a encore pris cette issue.
- Langage dominant
- R
- Étoiles
- 2.7k
- Forks
- 641
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
plotly depends on ggplot2 which alreadly has loaded function last_plot(), so, maybe we can throw it away as ggridge did
Brief description of the problem
library(plotly)
#> Loading required package: ggplot2
#>
#> Attaching package: 'plotly'
#> The following object is masked from 'package:ggplot2':
#>
#> last_plot
#> The following object is masked from 'package:stats':
#>
#> filter
#> The following object is masked from 'package:graphics':
#>
#> layout
plotly::last_plot
#> function ()
#> .store$get()
#> <bytecode: 0x7f9575ad7558>
#> <environment: namespace:plotly>
ggplot2::last_plot
#> function ()
#> .store$get()
#> <bytecode: 0x7f9575ad5cd8>
#> <environment: namespace:ggplot2>
Created on 2021-04-19 by the reprex package (v2.0.0)
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.
Piste de recherche
Commencez par comparer les points d’entrée publics plotly::last_plot et ggplot2::last_plot présentés dans le rapport, puis examinez comment plotly expose ou teste last_plot. Déterminez le comportement actuel du package et la couverture des tests avant de supprimer le doublon ; la tâche est terminée lorsque plotly ne masque plus le last_plot de ggplot2 sans interrompre l’utilisation prise en charge.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- r
- Domaine
- data-visualization
- Type d'issue
- Refactorisation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100