microsoft/Terminal

[Performance] Replace std::vector with more lightweight structure in `RenderData::GetPatternId`

Open

#10.564 aperta il 6 lug 2021

Vedi su GitHub
 (8 commenti) (1 reazione) (0 assegnatari)C++ (3212 fork)batch import
Area-PerformanceArea-RenderingHelp WantedIssue-TaskProduct-Meta

Metriche repository

Star
 (35.764 star)
Metriche merge PR
 (Merge medio 27g 19h) (24 PR mergiate in 30 g)

Descrizione

This is about the scrolling test in #10563. A simplified repro would be:

time bash -c 'yes | head -n1000000'

which takes 7-8s with windows terminal and 300ms with gnome terminal.

Test case provided by @lhecker.

These lines show up in the hot paths of the trace of OpenConsole:

https://github.com/microsoft/terminal/blob/main/src/renderer/base/renderer.cpp#L788

image

It appears that simply constructing empty std::vector (empty because this is in conhost/OpenConsole) is quite expensive.

CC @PankajBhojwani

Guida contributor

Direzione di ricerca
Ispeziona il codice in src/renderer/base/renderer.cpp alla riga 788 nel progetto OpenConsole. Comprendi perché costruire un std::vector vuoto è costoso nel percorso critico. Considera di sostituirlo con una struttura leggera come un array a dimensione fissa o un'ottimizzazione con buffer piccolo personalizzato. Esegui profilazione per verificare il miglioramento.
Tech stack
cpp
Dominio
desktop
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++STL containers
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40