glue-viz/glue-openspace

Avoid removing and adding graph node when updating properties

Aperta

#1 aperta il 26 giu 2019

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Python (4 fork)github user discovery
enhancementhelp wanted

Metriche repository

Star
 (2 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

At the moment, whenever any aspect of a layer is changed, we remove the graph node from OpenSpace and add a new one (this is done in layer_artist.py). We should make sure that we only add the graph node once and then make updates by setting only the properties that need to be set. This can include:

  • Changing the data to be shown (e.g. if a subset changes, or the attributes for the RA/Dec change)
  • Color
  • Visibility
  • Transparency
  • Point size/brightness

Guida contributor

Direzione di ricerca
Ispeziona l'implementazione corrente in layer artist.py per capire come i nodi del grafo vengono rimossi e aggiunti. Quindi identifica quali proprietà (dati, colore, visibilità, trasparenza, dimensione/luminosità dei punti) vengono aggiornate inutilmente e modifica il codice per impostare solo proprietà specifiche sul nodo esistente invece di ricrearlo.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70