Skip to content

Conversation

@JoshCu
Copy link
Member

@JoshCu JoshCu commented Apr 1, 2025

fix partitioning edge case with 2 partitions

[jcunningham8@compute001 NGIAB-HPCInfra]$ python test.py ../AWI_16_2863657_007/config/cat-2863657_subset.gpkg 1 .
Number of partitions: 1
Number of catchments: 241
Number of nexus: 122
Max cats per partition: 241
Actual number of partitions:
1
[jcunningham8@compute001 NGIAB-HPCInfra]$ python test.py ../AWI_16_2863657_007/config/cat-2863657_subset.gpkg 10 .
Number of partitions: 10
Number of catchments: 241
Number of nexus: 122
Max cats per partition: 25
Actual number of partitions:
10
[jcunningham8@compute001 NGIAB-HPCInfra]$ python test.py ../AWI_16_2863657_007/config/cat-2863657_subset.gpkg 100 .
Number of partitions: 100
Number of catchments: 241
Number of nexus: 122
Max cats per partition: 5
Actual number of partitions:
100
[jcunningham8@compute001 NGIAB-HPCInfra]$ python test.py ../AWI_16_2863657_007/config/cat-2863657_subset.gpkg 1000 .
Number of partitions: 122
Number of catchments: 241
Number of nexus: 122
Max cats per partition: 5
Actual number of partitions:
122
[jcunningham8@compute001 NGIAB-HPCInfra]$ python test.py ../AWI_16_2863657_007/config/cat-2863657_subset.gpkg 2 .
Number of partitions: 2
Number of catchments: 241
Number of nexus: 122
Max cats per partition: 121
Actual number of partitions:
2

fixes issues in CIROH-UA/NGIAB-HPCInfra#39

fix partitioning edge case with 2 partitions
@JoshCu JoshCu requested a review from benlee0423 April 1, 2025 17:22
@benlee0423
Copy link

I will get it tested on Pantarhei, and update it
Thank you for the quick fix.

@benlee0423 benlee0423 merged commit 6932f1c into ngiab Apr 1, 2025
@JoshCu JoshCu deleted the partition-patch branch April 24, 2025 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants