perf: renderer highlights via vim.hl.range are slow on large trees
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 78/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- lua
- Lĩnh vực
- frontend, performance
Hướng nghiên cứu
Bắt đầu từ Renderer:render_hl và theo dõi cách nó áp dụng các phạm vi byte trên một dòng cùng các mức ưu tiên. So sánh các lệnh gọi vim.hl.range hiện có với nvim_buf_set_extmark khi dùng vim.hl.priorities.user; xác minh các phần tô sáng thu được là giống hệt nhau và đo thời gian kết xuất trên một cây lớn. Công việc hoàn tất khi các lệnh gọi extmark trực tiếp giữ nguyên phạm vi, mức ưu tiên và giao diện, đồng thời giảm thời gian kết xuất.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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).
- Ngôn ngữ chính
- Lua
- Star
- 8.7k
- Fork
- 642
- Merge trung bình
- 1 ngày 4 phút
- Pull request đã merge (30 ngày)
- 11
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của nvim-tree/nvim-tree.lua
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
nvim-tree/nvim-tree.lua#3383 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/100
nvim-tree/nvim-tree.lua#3382 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Use vim.Option getters and setters instead of directly settingCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
nvim-tree/nvim-tree.lua#3357 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Migrate To vim.packĐang mởdocumentation PR please
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 65/100
nvim-tree/nvim-tree.lua#3179 · 6 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
perf: Filters:custom calls getcwd for every file, even with no custom filtersCó thể đã có người làm @hood đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 25/100
nvim-tree/nvim-tree.lua#3386 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của nvim-tree/nvim-tree.lua
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug confirmed discord guide:tyrs-watch locale:enUS navigation v0.3.2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
fitzforge/polestar#855 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
translation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 64/100
KristanLaimon/PanelsPlus#16 ·
-
Bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 91/100
DennisRas/WeeklyKnowledge#164 ·
-
linux 上使用 msvc-wine 无法增量构建Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
Maintainer thường phản hồi trong vòng 1 ngày