plotly / plotly/plotly.js

Unclear Intended behavior for marker.size=0 for scatter and scattergl

Offen
#6,862 0 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@LiamConnors arbeitet bereits daran.

Seit 12.7.2024.

bug P3
Vorherrschende Sprache
JavaScript
Sterne
18.3k
Forks
2k
Ø Merge
2 T. 12 Std.
Gemergte PRs (30 T.)
28

Beschreibung

Setting the marker size to zero could have two reasonable interpretations:

  1. 0 is a valid input and the markers don't get plotted
  2. 0 is not a valid input and the markers get plotted in the default size

It seems Option 2 is the current implementation as of v2.28.0 (codepen link). But is that intentional? The documentation says

size
Parent: data[type=scattergl].marker
Type: number or array of numbers greater than or equal to 0
Default: 6
Sets the marker size(in px).

The documentation sounds a bit more like option 1, to me, because 0 is inside the valid range. Also, an array of all 0s does not get plotted, which I assumed should be the same behavior of a single 0.

Should the documentation be clarified or the implementation be changed?

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.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

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