Microsoft/vscode

Searching in terminal with large scrollback and many wrapped lines is slow

Open

#152.548 geöffnet am 18. Juni 2022

Auf GitHub ansehen
 (4 Kommentare) (2 Reaktionen) (1 zugewiesene Person)TypeScript (10.221 Forks)batch import
bughelp wantedperfterminal-findupstreamupstream-issue-linked

Repository-Metriken

Stars
 (74.848 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 11h 43m) (1.000 gemergte PRs in 30 T)

Beschreibung

Might be similar to #147685.

macOS: 12.4 (21F79) code:

Version: 1.68.1 (Universal)
Commit: 30d9c6cd9483b2cc586687151bcbcd635f373630
Date: 2022-06-14T12:52:13.188Z
Electron: 17.4.7
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: 9.8.177.13-electron.0
OS: Darwin arm64 21.5.0

Around 2 million characters. But extremely fast on my native terminal.

https://user-images.githubusercontent.com/16152305/174429543-b3b91608-7132-447c-a3cd-08ca765ce186.mov

Contributor Guide

Research-Richtung
Untersuchen Sie die Implementierung der Terminal Suche in der vscode Codebasis, wahrscheinlich in src/vs/workbench/parts/terminal. Siehe Issue #147685 für ähnliche Leistungsverbesserungen. Profilieren Sie den Suchvorgang mit großem Scrollback mithilfe des integrierten Profilers oder Chrome DevTools, um Engpässe zu identifizieren. Konzentrieren Sie sich auf das Rendern von umbrochenen Zeilen und die Handhabung des Textpuffers.
Tech Stack
typescript
Domain
performance
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptVS Code extension development
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40