Refactor updateUnknownsAndDuplicates to only check certain wells, making it more efficient
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
- Tipo di issue
- Refactoring
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- frontend, performance
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.
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—
Probably.
updateUnknownsAndDuplicatesis 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?
- Lingua principale
- Python
- Stelle
- 2
- Fork
- 15
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di biocore/LabControl
-
bug front-end question
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
biocore/LabControl#594 ·
-
front-end question
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
biocore/LabControl#593 ·
-
Cache list of active samples when the active study is changed in the plating interfaceForse di nuovo libera @fedarko l’ha presa 2581 giorni fa e non c’è nessuna pull request aperta. Apertafront-end
biocore/LabControl#592 · 1 assegnatario ·
-
priority:low
Difficoltà 3/5 1-2 giorni Idoneità per principianti 20/100
biocore/LabControl#590 ·
-
priority:low scope:small
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
biocore/LabControl#589 · 2 commenti ·
Tutte le issue di biocore/LabControl
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 Mezza giornata Idoneità per principianti 70/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
Qiskit/qiskit-ibm-runtime#3431 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
[Lesson] A compatibility-gate rejection is a verdict, not something to overwrite with --accept-riskApertalesson-submission needs-ac pending-review
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
Ikalus1988/MisakaNet#2870 ·
I maintainer di solito rispondono entro 1 giorno
-
feature:LinkChecker
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
digitalfabrik/integreat-cms#4594 ·
I maintainer di solito rispondono entro 5 giorni