Consider replacing attrs with msgspec
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- python
- Lĩnh vực
- cli, performance
Hướng nghiên cứu
Bắt đầu bằng cách xác định các dataclass của attrs được sử dụng bởi các đường dẫn CLI và xử lý dataset; issue không nêu tên tệp hoặc entry point cụ thể nào. Đo độ phản hồi khi import CLI và tổng thời gian chạy trên một dataset lớn, sau đó so sánh attrs với msgspec.Struct và ghi lại liệu mức cải thiện hiệu năng có xứng đáng với thay đổi về khả năng đọc hay không.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
As this project gets closer to usability, performance on large datasets is going to become relevant. I suspect that attrs is not a bottleneck, but we do instantiate a large number of attrs dataclasses. We should evaluate whether switching to msgspec.Struct offers nontrivial performance improvements.
I would say the main two relevant measures would be:
- CLI responsiveness. If importing is slow the CLI will feel sluggish, even if we're talking a one-time cost of a couple hundred ms.
- Total runtime on a large dataset. 5-10% speedup is probably worth it. <5% if the code becomes simpler to read.
- Ngôn ngữ chính
- Python
- Star
- 10
- Fork
- 7
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
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 bids-standard/python-validator
-
Add a structured issues model and emit findings from the filename validatorCó thể đã có người làm @karellopez đã nhận 81 ngày trước. Đang mởenhancement
bids-standard/python-validator#85 · 1 bình luận · 1 người được giao ·
-
Populate associations contextCó thể làm lại được @ubdbra001 đã nhận 115 ngày trước và không có pull request nào đang mở. Đang mở
bids-standard/python-validator#82 · 1 bình luận · 1 người được giao ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
-
Validate DerivativesCó thể đã có người làm @bendhouseart đã nhận 298 ngày trước. Đang mởenhancement
bids-standard/python-validator#62 · 1 người được giao ·
-
Populate tiff contextĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
Tất cả issue của bids-standard/python-validator
Issue tương tự
-
getchaintips leaves out the active tip once a header is staged on it, where Core always lists itĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
btclib-org/btclib-node#2158 ·
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 85/100
dbt-msft/dbt-sqlserver#897 ·
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 65/100
Gathin23/cleanyourtv#2770 · 2 bình luận ·
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 68/100
vllm-project/vllm#64845 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
dekobon/big-code-analysis#1729 ·
Maintainer thường phản hồi trong vòng 1 ngày