Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

#487: Introduce additional filters prior to subclustering #517

Merged
merged 5 commits into from
Sep 28, 2024
Merged

#487: Introduce additional filters prior to subclustering #517

merged 5 commits into from
Sep 28, 2024

Conversation

cwschilly
Copy link
Contributor

Fixes #487

@cwschilly cwschilly linked an issue Jun 13, 2024 that may be closed by this pull request
Introduce additional additional filters prior to subclustering #487
Closed
3 tasks
@cwschilly cwschilly marked this pull request as draft June 13, 2024 20:09
@cwschilly cwschilly self-assigned this Jun 13, 2024
@cwschilly cwschilly force-pushed the 487-introduce-additional-additional-filters-prior-to-subclustering branch from 259e2ca to 728090a Compare September 18, 2024 19:04
@cwschilly cwschilly force-pushed the 487-introduce-additional-additional-filters-prior-to-subclustering branch from 786084d to 197d8b7 Compare September 23, 2024 14:02
@cwschilly cwschilly marked this pull request as ready for review September 24, 2024 13:49
@cwschilly cwschilly merged commit c52fc42 into develop Sep 28, 2024
10 checks passed
@cwschilly cwschilly deleted the 487-introduce-additional-additional-filters-prior-to-subclustering branch September 28, 2024 14:49
github-actions bot pushed a commit that referenced this pull request Sep 28, 2024
@cwschilly
#487: Introduce additional filters prior to subclustering (#517)
4f9a07a 
* #487: add filters to subclustering based on load before and after

* #487: fix failing test, set default thresholds to 0.0

* #487: make subclustering filters configurable

* #487: fix pylint warnings

* #487: fix formatting of max_load
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ppebay ppebay ppebay approved these changes

@lifflander lifflander Awaiting requested review from lifflander

@nlslatt nlslatt Awaiting requested review from nlslatt

@tlamonthezie tlamonthezie Awaiting requested review from tlamonthezie

Assignees

@cwschilly cwschilly

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Introduce additional additional filters prior to subclustering
2 participants
@cwschilly @ppebay