plotly / plotly/plotly.js

[BUG]: please provide an unbundled version of stackgl_modules

Offen
#7,654 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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

Beschreibung

Description

I use a very minimal configuration consisting of just a few modules:

import * as Plotly from "plotly.js/lib/core.js";
import * as scatter from "plotly.js/lib/scatter";
import * as surface from "plotly.js/lib/surface";
Plotly.register([scatter, surface]);

the via node_modules/plotly.js/src/traces/surface/convert.js, line 3, this pulls in stackgl_modules:

var createSurface = require('../../../stackgl_modules').gl_surface3d;

As a result, the whole stackgl_modules/index.js file is pulled in, even though only a few parts are needed.

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 Untersuchung von src/traces/surface/convert.js und der referenzierten stackgl_modules/index.js, um nachzuverfolgen, warum der Import des surface trace das gesamte Modul einbindet. Bestimme den Umfang einer ungebündelten stackgl_modules-Schnittstelle, die das minimale Plotly-Registrierungsbeispiel beibehält, und überprüfe anschließend, dass nur die angeforderten surface-Abhängigkeiten enthalten sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
data-visualization
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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