plotly / plotly/plotly.R

add_ribbons can't handle NA values

Offen
#1,884 1 Kommentar 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
R
Sterne
2.7k
Forks
641
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

The plotly function add_ribbons() doesn't function properly when cases in the middle or at the end have NA values. These lead to a weird behaviour where parts of the ribbons are missing or drawn in wrong places (see Plots 1 & 3). When the connectgaps-Attribute is set to TRUE for the ribbon the bug doesn't appear (see Plot 2).
NA values in the beginning don't seem to be a problem as well (see Plot 3, Red Trace 2).

The same behaviour has also been reported with ggplot and plotly in issue #1060. But because it may either be two separate problems or just a problem in plotly itself also affecting the ggplot visualization I decided to open this separate issue.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit der R-Funktion add_ribbons() und reproduziere das Verhalten anhand der verlinkten Plots 1–3, wobei der Fokus auf NA-Werten in der Mitte und am Ende liegt. Vergleiche das Ergebnis mit connectgaps auf TRUE gesetzt und mit dem verwandten ggplot- und plotly-Issue #1060; als erledigt gilt die Aufgabe, wenn die Ribbons an den richtigen Stellen gerendert werden, ohne connectgaps zu benötigen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
r
Bereich
data-visualization
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.