Map parameterization should be reverted
Maintainer thường phản hồi trong vòng 2 ngày
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
- 25/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- cpp
- Lĩnh vực
- computer-graphics
Hướng nghiên cứu
Bắt đầu bằng việc xem xét commit c4aabc570103e72876e9889a20396e719f52bca4 và đường dẫn renderd áp dụng parameterize_function cho các đối tượng bản đồ. So sánh hành vi đó với những mối lo ngại trong các issue 2965 và 2553 của Mapnik. Được xem là hoàn tất khi việc đột biến bản đồ trong mỗi lần render được hoàn tác hoặc thay thế bằng một thiết kế đã thống nhất, đồng thời không có bản sao bản đồ không cần thiết khi không sử dụng tham số hóa.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
/cc @jburgess777
Looking into ttps://github.com/mapnik/mapnik/issues/2965, I wondered if renderd was doing anything unsafe.
I noticed that https://github.com/openstreetmap/mod_tile/commit/c4aabc570103e72876e9889a20396e719f52bca4 added mutation of map objects per render. While this might work I think it should be reverted for at least these reasons:
- A copy of the
mapnik.Mapis made every time a tile is rendered even if noparameterize_functionis set. This is potentially very expensive. - I think parameterization would be much better done in XML before loading the map. Ideally maps should not be modified after being loaded.
- If per render changes to styling truly need to be made, then a much better design would be to leverage Mapnik 3.x variables: https://github.com/mapnik/mapnik/issues/2553
- Ngôn ngữ chính
- C++
- Star
- 318
- Fork
- 200
- Merge trung bình
- 2 ngày 22 giờ
- Pull request đã merge (30 ngày)
- 22
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 openstreetmap/mod_tile
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 65/100
openstreetmap/mod_tile#292 · 4 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
openstreetmap/mod_tile#477 · 7 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
openstreetmap/mod_tile#466 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
openstreetmap/mod_tile#462 · 7 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 30/100
openstreetmap/mod_tile#447 · 3 bình luận · 1 reaction ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của openstreetmap/mod_tile
Issue tương tự
-
launch-client --save-creds creates creds/<account>/ with mode 755, not 700Có thể đã có người làm @metis-sprock đã nhận hôm nay. Đang mởbacklog bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
EchoTools/nevr-runtime#1471 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
.NET ep:WebGPU platform:web
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
microsoft/onnxruntime#33755 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
grumpycoders/pcsx-redux#2530 ·
-
A-tools P-medium S-ready-to-start studio task
Độ 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
-
Status: Needs confirmation Status: Needs triage Type: Code Quality
Độ 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