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

Add clustering algorithms #1963

Closed
maxkfranz opened this issue Sep 5, 2017 · 1 comment

Comments

Projects
None yet
1 participant
@maxkfranz
Copy link
Member

commented Sep 5, 2017

Merge the following (unpublished to npm) extensions to the main lib:

For each:

  • Review code
    • High-level review for more obvious issues
    • In-depth review
  • Merge source
  • Merge tests
  • Port readme docs to main lib docs

@maxkfranz maxkfranz added this to the 3.3.0 milestone Sep 5, 2017

@maxkfranz maxkfranz closed this Oct 27, 2017

@maxkfranz maxkfranz reopened this Nov 10, 2017

@maxkfranz

This comment has been minimized.

Copy link
Member Author

commented Nov 10, 2017

Fuzzy c-means has an issue that is erratic: https://travis-ci.org/cytoscape/cytoscape.js/jobs/300396692

maxkfranz added a commit that referenced this issue Apr 16, 2018

Rework the fuzzy c-means test that sometimes fails #1963
- Increase the number of tries to increase reproducibility.
- Simplify the clusters comparison to a human-readable string comparison.
- Increasing the distance between the expected clusters seems to make the issue not reproducible.  Perhaps it's just much less likely.
- The issue seems to stem from cases where the centroids are much different from where the test implicitly expects.  This happens rarely.

@maxkfranz maxkfranz closed this May 4, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.