perf: renderer highlights via vim.hl.range are slow on large trees
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- lua
- Área
- frontend, performance
Línea de trabajo
Empieza en Renderer:render_hl y sigue cómo aplica rangos de bytes de una sola línea y prioridades. Compara las llamadas existentes a vim.hl.range con nvim_buf_set_extmark usando vim.hl.priorities.user; verifica que los resaltados resultantes sean idénticos y mide el tiempo de renderizado en un árbol grande. Se considera terminado cuando las llamadas directas a extmark conservan los mismos rangos, prioridades y apariencia, y reducen el tiempo de renderizado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
Renderer:render_hl applies every highlight of every line with vim.hl.range. That function is generic: on each call it switches buffer context with vim._with, calls col('$') twice and calls getregionpos. nvim-tree's ranges are always single-line byte ranges, so none of that work is needed.
Impact
| µs per highlight | |
|---|---|
vim.hl.range |
6.1 |
nvim_buf_set_extmark |
0.74 |
At 16.8k visible lines, render_hl is about 50% of Renderer:draw() (620ms) and about 15% of reload_explorer.
Proposed fix
Call nvim_buf_set_extmark directly with the same ranges and the same priority (vim.hl.priorities.user). The highlights produced are identical. Happy to provide a PR for this ofc.
Environment: macOS, Nvim v0.13.0-dev, LuaJIT 2.1. Test repo: 16.8k files and folders, 3.1k git status entries (modified, untracked, ignored). Measured headless with vim.uv.hrtime() and the LuaJIT profiler (jit.p).
- Lenguaje dominante
- Lua
- Estrellas
- 8.7k
- Forks
- 642
- Merge medio
- 1 d 4 min
- PR fusionados (30 d)
- 11
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nvim-tree/nvim-tree.lua
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
nvim-tree/nvim-tree.lua#3383 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
nvim-tree/nvim-tree.lua#3382 ·
Los mantenedores suelen responder en 1 día
-
Use vim.Option getters and setters instead of directly settingPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
nvim-tree/nvim-tree.lua#3357 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Migrate To vim.packAbiertodocumentation PR please
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
nvim-tree/nvim-tree.lua#3179 · 6 comentarios ·
Los mantenedores suelen responder en 1 día
-
perf: Filters:custom calls getcwd for every file, even with no custom filtersPosiblemente ocupada @hood la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 25/100
nvim-tree/nvim-tree.lua#3386 ·
Los mantenedores suelen responder en 1 día
Todos los issues de nvim-tree/nvim-tree.lua
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
Los mantenedores suelen responder en 1 día
-
bug confirmed discord guide:tyrs-watch locale:enUS navigation v0.3.2
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
fitzforge/polestar#855 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
translation
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
KristanLaimon/PanelsPlus#16 ·
-
Bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
DennisRas/WeeklyKnowledge#164 ·
-
linux 上使用 msvc-wine 无法增量构建Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día