fossasia/visdom

Properties box not updating

Chiusa

#693 aperta il 11 dic 2019

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (1 assegnatario)Python (1152 fork)batch import
bughelp wanted

Metriche repository

Star
 (9831 stelle)
Metriche merge PR
 (Merge medio 9g 12h) (116 PR mergiate in 30 g)

Descrizione

Bug Description When I trigger an update with viz.properties(), the window does not update.

Reproduction Steps Enter steps to reproduce the behavior:

  1. In python, run viz.properties([...], win='mywin')
  2. Open visdom in the browser
  3. Update the window with viz.properties([...], win='mywin')
  4. The property window does not update
  5. Refresh the browser. The property window contains the correct data.

Expected behavior The property window should update with new values.

Screenshots N/A

Client logs: The javascript console shows a warning:

main.js?v=47fc3afa811b358b354df09184e392e7:19 Warning: validateDOMNesting(...): <tr> cannot appear as a child of <table>. Add a <tbody> to your code to match the DOM tree generated by the browser.

However, I tested a patched version with <tbody> and it still had the update bug, even though the warning went away. So it looks like this not an issue.

I added some console.log statements inside of the render() functions, and it looks like the pane is re-rendering with the correct values. Somehow that update is not being reflected in the DOM.

Server logs: N/A

Additional context Using Chrome 78.0.3904.108 on Mac. Using visdom 0.1.8.9

Guida contributor

Direzione di ricerca
Indaga la funzione di render di React per il pannello delle proprietà in main.js. Il bug è probabilmente correlato a come il componente aggiorna il suo stato o a come il DOM viene riconciliato. Controlla se manca una prop key o se c'è un uso improprio dello stato. Verifica anche se il problema persiste con versioni più recenti di React o se è un problema di compatibilità con il browser. Esamina l'avviso della console riguardante l'annidamento di <tr>, anche se l'aggiunta di <tbody> non ha risolto l'aggiornamento.
Tech stack
pythonjavascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Basic React knowledgeJavaScript debugging
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50