Skip to content
This repository was archived by the owner on Nov 13, 2025. It is now read-only.

Conversation

@Aleksa-M
Copy link
Contributor

No description provided.

Copy link
Contributor

@mgupta27 mgupta27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed.

Copy link
Member

@maxlou05 maxlou05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reviewed

config.yaml Outdated
cluster_estimation:
min_activation_threshold: 25
min_new_points_to_run: 5
random_state: 0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a newline to the end here

Copy link
Contributor

@mgupta27 mgupta27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed.

Copy link
Contributor

@mgupta27 mgupta27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Aleksa-M Aleksa-M merged commit fe018af into main Nov 14, 2024
1 check passed
@Aleksa-M Aleksa-M deleted the cluster-estimation-integration-redo-redo branch November 14, 2024 21:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants