how about removing function last_plot?
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- R
- Sterne
- 2.7k
- Forks
- 641
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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)
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die im Bericht gezeigten öffentlichen Einstiegspunkte plotly::last_plot und ggplot2::last_plot zu vergleichen, und untersuche anschließend, wie plotly last_plot bereitstellt oder testet. Ermittle das bestehende Paketverhalten und die Testabdeckung, bevor du das Duplikat entfernst; abgeschlossen ist die Aufgabe, wenn plotly last_plot von ggplot2 nicht mehr maskiert, ohne die unterstützte Verwendung zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- r
- Bereich
- data-visualization
- Issue-Typ
- Refactoring
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100