scverse / scverse/spatialdata-plot

consider removing `_copy` private method

Offen
#47 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

enhancement priority: low
Vorherrschende Sprache
Python
Sterne
86
Forks
21
Ø Merge
14 Std. 50 Min.
Gemergte PRs (30 T.)
3

Beschreibung

Report

https://github.com/scverse/spatialdata-plot/blob/582c8379f2478d0e7bc0a7d4733d3cb6fa9af061/src/spatialdata_plot/pl/basic.py#L81

copying the entire spatialdata object can be expensive if elements are on memory. Consider change logic to avoid this.

Version information

No response

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 damit, src/spatialdata_plot/pl/basic.py ungefähr ab Zeile 81 zu lesen und nachzuverfolgen, wo die private _copy-Methode verwendet wird. Untersuche, ob das Plotten das Kopieren des gesamten spatialdata-Objekts vermeiden kann, wenn sich die Elemente im Speicher befinden. Als abgeschlossen gilt die Aufgabe, wenn das bestehende Plotting-Verhalten erhalten bleibt und gleichzeitig das teure Kopieren des vollständigen Objekts vermieden wird; in der Issue wird keine Testdatei genannt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
data-visualization
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
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.