plotly / plotly/plotly.R

crosstalk equality

Aperta
#1,167 4 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Lingua principale
R
Stelle
2.7k
Fork
641
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

I discovered this as I worked through an example of linking leaflet with DT and plotly through crosstalk (example). It seems plotly converts crosstalk keys to String and then uses exact equality match .indexOf (see lines). Thinking through how best to solve this, I came up with the following:

  1. Not convert crosstalk key to string in plotly, but I am guessing there was a reason to do the conversion for plotly traces.

  2. Perform match using something other than indexOf or ===. For the purposes of my example I used [].filter, but this is inefficient.

  3. Expect other htmlwidget authors to also convert to string, or add both the string and non-string forms to crosstalk select.

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Riproduci l’esempio crosstalk collegato in experiments/select_crosstalk.R ed esamina la logica di corrispondenza a cui si fa riferimento alla riga 739 di plotly.js. Confronta le rappresentazioni delle chiavi utilizzate da leaflet, DT e plotly, quindi stabilisci quale comportamento di corrispondenza dovrebbe essere coerente in tutto l’esempio prima di modificare qualsiasi cosa. Il lavoro è concluso quando i widget collegati selezionano in modo affidabile le stesse chiavi.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript, r
Ambito
data-visualization
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.