biocore / biocore/LabControl

Refactor updateUnknownsAndDuplicates to only check certain wells, making it more efficient

Aperta
#591 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
code refactor front-end
Lingua principale
Python
Stelle
2
Fork
15
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Recapping this issue: I wrote the following comment in `patchWell()`—
https://github.com/biocore/LabControl/blob/10ef1ef11f677957f23dc9765235e5642a81e1e1/labcontrol/gui/static/js/plateViewer.js#L458-L472

@AmandaBirmingham responded to it [here](https://github.com/biocore/LabControl/pull/585#discussion_r322494558)—

>Probably. `updateUnknownsAndDuplicates` is completely agnostic to whatever change triggered it--but that means it has to be brute-force and re-check everything. I think it is the source of a lot of the performance issues in the platemapper, and probably needs to be rethought from the ground up. Could you add an issue to the repo for this, and then remove this comment?

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia in labcontrol/gui/static/js/plateViewer.js, in patchWell() e updateUnknownsAndDuplicates(), poi leggi la discussione collegata di PR #585 per il contesto sulle prestazioni. Traccia quali wells può influenzare ogni modifica e definisci come il check refactorizzato li individuerà. Il lavoro è completato quando la funzione non controlla più con la forza bruta i wells non correlati, mantenendo al contempo il rilevamento dei wells sconosciuti e duplicati.

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

Valutazione

Stack tecnologico
javascript
Ambito
frontend, performance
Tipo di issue
Refactoring
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.