vgsvg enhancement: avoid transform instructions in svg code
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- go
- Lĩnh vực
- data-visualization
Hướng nghiên cứu
Bắt đầu bằng cách xác định mã tạo SVG chịu trách nhiệm cho các thuộc tính biến đổi của nhóm và văn bản, sau đó theo dõi cách các tọa độ biểu đồ được chuyển đổi trước khi xuất SVG. So sánh SVG được tạo ra khi có và không có các phép biến đổi lặp lại, đồng thời xác định các kiểm thử kết xuất liên quan, nếu có. Hoàn tất khi các biểu đồ tương đương được tạo ra sau khi loại bỏ các chỉ thị biến đổi dư thừa và đầu ra SVG nhỏ hơn.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Suggestion
To make generated svg code more compact, I suggest to make the transform in the Go code instead of the svg code.
The change would allow to change
<g transform="scale(1, -1) translate(0, -216)">into<g>which occur once in the svg code.<text x="462.81" y="-0.2002" transform="scale(1, -1)" style=...into<text x="462.81" y="-0.2002" style=...which occur for every text tag in the svg code.
Benefits :
- reduce the size of the SVG code for plots,
- execute the transform once at svg code generation, not at each plot rendering.
Disadvantages :
- make the go code a little more complex by adding the transform instruction
Another optimization could be to use style definition in the svg code to avoid repeating the style in each text tag. This would be more beneficial to the size, but also slightly more complex to implement.
If you agree, I may work on a PR proposal.
The use case I have is that I attach multiple svg plots to mails which has a maximum size limit. Whoever would want to save/archive many plots would be glad that they are small. They are already smaller than PNG plots, but we could do better than that.
- Ngôn ngữ chính
- Go
- Star
- 3k
- Fork
- 202
- 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
- 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 gonum/plot
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 52/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 32/100
-
plot/plotter: NewHist(binPoints) panic when a small negative number is usedCó thể đã có người làm @SashaMIT đã nhận 22 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Maintainer thường phản hồi trong vòng 1 ngày
-
agent-butler-finding chore
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 78/100
jordansmall/spindrift#6318 ·
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 72/100
Eyevinn/moqlivemock#166 ·
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 72/100
JakeChampion/lang#14091 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
batcher: BAF send context leaks per batchCó 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 84/100
hyperledger/fabric-x-orderer#1409 ·
Maintainer thường phản hồi trong vòng 3 ngày