plotly / plotly/plotly.R

how about removing function last_plot?

Ouverte
#1,943 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

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

  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.

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

Recevez les nouvelles issues par e-mail

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