fossasia/visdom

Properties box not updating

Fermée

#693 ouverte le 11 déc. 2019

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (1 personne assignée)Python (1 152 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (9 831 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Examinez la fonction de rendu React pour le panneau des propriétés dans main.js. Le bug est probablement lié à la façon dont le composant met à jour son état ou à la manière dont le DOM est réconcilié. Vérifiez s'il manque une prop key ou si l'état est utilisé de manière incorrecte. Vérifiez également si le problème persiste avec les versions plus récentes de React ou s'il s'agit d'un problème de compatibilité avec le navigateur. Consultez l'avertissement de la console concernant l'imbrication de <tr>, même si l'ajout de <tbody> n'a pas corrigé la mise à jour.
Stack technique
pythonjavascriptreact
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Basic React knowledgeJavaScript debugging
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50