matplotlib / matplotlib/ipympl
Optimizations
Nessuno ha ancora preso questa issue.
- Lingua principale
- Jupyter Notebook
- Stelle
- 1.7k
- Fork
- 234
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
I feel like there could be some optimizations concerning the image update.
1. We should draw binary data directly on the canvas using:
```javascript
context.putImageData(binary_data);
```
instead of creating a Blob that is png encoded. That would also allow the back-end to not waste time encoding into png (if that is possible).
2. I feel like the "diff" computation should actually be slower than sending the full image, as it needs an extra computation of "which pixels actually needs a redraw?": https://github.com/matplotlib/matplotlib/blob/5fc44eea64a57ab1c6d236cf7c71906c9d40e6f6/lib/matplotlib/backends/backend_webagg_core.py#L190-L195.
Plus, we are still sending a full image. Maybe the png encoding does some compression if lots of pixels are transparent? If that is not the case, we are still sending the same amount of data.
Guida per i contributori
Apri la guida per i contributori
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.
Direzione di ricerca
Inizia dal codice di aggiornamento delle immagini referenziato in matplotlib/backends/backend_webagg_core.py e segui il corrispondente punto di ingresso per l’aggiornamento del canvas in ipympl. Confronta il percorso proposto per i dati binari con gli attuali percorsi PNG e diff e definisci benchmark e criteri di accettazione prima di modificare uno dei due lati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript, python
- Ambito
- backend, 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