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

Change KMeans init algorithm to one with less skew #181

Closed
wants to merge 17 commits into
base: svn-trunk
from

Conversation

Projects
None yet
2 participants
@Komzpa
Copy link
Member

Komzpa commented Jan 4, 2018

No description provided.

Komzpa added some commits Jan 4, 2018

@codecov

This comment has been minimized.

Copy link

codecov bot commented Jan 4, 2018

Codecov Report

Merging #181 into svn-trunk will increase coverage by 0.13%.
The diff coverage is 90%.

Impacted file tree graph

@@              Coverage Diff              @@
##           svn-trunk     #181      +/-   ##
=============================================
+ Coverage      79.22%   79.35%   +0.13%     
=============================================
  Files            202      202              
  Lines          63011    63616     +605     
=============================================
+ Hits           49921    50484     +563     
- Misses         13090    13132      +42
Impacted Files Coverage Δ
liblwgeom/lwkmeans.c 83.16% <90%> (-2.55%) ⬇️
liblwgeom/lwgeom_geos_split.c 68.61% <0%> (-1.95%) ⬇️
liblwgeom/lwgeom_median.c 98.7% <0%> (-1.3%) ⬇️
liblwgeom/lwcollection.c 84.88% <0%> (-0.57%) ⬇️
liblwgeom/ptarray.c 91.27% <0%> (-0.35%) ⬇️
liblwgeom/lwgeom_api.c 69.64% <0%> (-0.04%) ⬇️
liblwgeom/cunit/cu_algorithm.c 99.07% <0%> (-0.04%) ⬇️
liblwgeom/lwin_wkt.c 81% <0%> (+0.26%) ⬆️
liblwgeom/lwout_x3d.c 74.46% <0%> (+5.52%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c638b40...47214b7. Read the comment docs.

@Komzpa

This comment has been minimized.

Copy link
Member Author

Komzpa commented Jan 5, 2018

Komzpa added some commits Jan 8, 2018

@pramsey

This comment has been minimized.

Copy link
Member

pramsey commented Jan 9, 2018

I read through it, seems nice to me.

@strk strk closed this in e8568a8 Jan 9, 2018

@Komzpa Komzpa deleted the kmeans_init branch Jan 9, 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.