Refactor updateUnknownsAndDuplicates to only check certain wells, making it more efficient
- 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