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.

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

Implement the Chinese Whispers clustering algorithm in PHP code. #691

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

matiasdelellis
Copy link
Owner

@matiasdelellis matiasdelellis commented Aug 22, 2023

To the happiness of many (Issue #690, #688, #687, #685, #649, #632, #627, #625, etc..?) this means that we do not depend on the pdlib extension, but it goes without saying that its use is still highly recommended.

You will understand that it is slower, however I must admit that with JIT enabled, it is quite acceptable, and this is the only reason why decided to publish it.

It is still experimental, and it works, but it has problems such as it seems not to converge in stable clusters. When I can fix this, it will probably be even slower.

@matiasdelellis matiasdelellis force-pushed the native-cwc branch 3 times, most recently from 1f80f52 to 8854a1f Compare August 22, 2023 23:14
To the happiness of many (Issues #690, #688, #687, #685, #649, #632
, #627, #625, etc..?) this means that we do not depend on the pdlib
extension, but it goes without saying that its use is still highly
recommended.

You will understand that it is slower, however I must admit that with
JIT enabled, it is quite acceptable, and this is the only reason why
decided to publish it.
@matiasdelellis matiasdelellis merged commit 4ad739e into master Aug 23, 2023
@matiasdelellis matiasdelellis deleted the native-cwc branch August 28, 2023 13:50
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.

1 participant