Issues to address in pgr_contractionHierarchies before promotion from experimental status
Maintainer thường phản hồi trong vòng 2 ngày
@cvvergara đang làm issue này rồi.
Từ ngày 25/4/2025.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Issues to Address for pgr_contractionHierarchies
This issue tracks the remaining work needed for PR #2868: New function pgr contraction hierarchies 3.8 by @cvvergara before the pgr_contractionHierarchies function can be promoted from experimental to proposed status.
Background
The pgr_contractionHierarchies function was added as experimental as it still has various issues that need to be resolved. As noted in the PR discussion:
The function is added as experimental as it still has a lot of issues, some are noticed by the rabbit, others are for code organization. When the issues are fixed then we can promote to proposed.
Issues to Address
Based on the PR reviews and comments, the following issues need to be addressed:
Code Organization Issues
- Review and improve the organization of code files
- Ensure proper code structure for maintainability
- Apply consistent coding style across implementation
CodeRabbit-Identified Issues
- Fix potential memory leak in
result_tupleswhenSRF_RETURN_DONEis reached - Review and fix issues with negative
reverse_costin shortcut edges - Fix template argument from visitor instantiation in contractionHierarchies.hpp
- Guard against empty priority-queue before calling
top() - Review parameter passing in no_crash_test.pg
- Consider adding metric values for shortcut edges
Documentation & Tests
- Ensure all test plans accurately match the number of assertions
- Update documentation to clearly explain the function's experimental status and caveats
- Add additional examples showing integration with other algorithms
Action Plan
- Address code organization issues first
- Fix implementation bugs identified by code review
- Improve tests and documentation
- Verify all issues are resolved
- Promote the function from experimental to proposed status
References
- Ngôn ngữ chính
- C++
- Star
- 1.4k
- Fork
- 397
- Merge trung bình
- 2 ngày 2 giờ
- Pull request đã merge (30 ngày)
- 4
Chuẩn bị môi trường
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 pgRouting/pgrouting
-
Boost broken linksCó thể đã có người làm @Nandan-Raichura-Dev đã nhận 1 ngày trước. Đang mởEnhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
pgRouting/pgrouting#3176 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Add details to CONTRIBUTING.mdĐang mởAdministrative Enhancement
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 80/100
Maintainer thường phản hồi trong vòng 2 ngày
-
points_input Developers docCó thể đã có người làm @withaman69 đã nhận 86 ngày trước. Đang mởC/C++ Developers Doc. good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
pgRouting/pgrouting#973 · 10 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
get_check_data developers docCó thể đã có người làm @withaman69 đã nhận 88 ngày trước. Đang mởC/C++ Developers Doc. good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
pgRouting/pgrouting#972 · 7 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
circuits Functionality Request
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của pgRouting/pgrouting
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
elf-alchemist/elfbsp#113 ·
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 75/100
patrick-hudson/put.gd-desktop#979 · 1 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 76/100
Dao-AILab/flash-attention#3097 ·
Maintainer thường phản hồi trong vòng 3 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
Niko1221/Strata#4779 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 3 ngày
-
tech-debt
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
prestonbrown/helixscreen#1990 ·
Maintainer thường phản hồi trong vòng 1 ngày