search

StevenReitsma / kaggle-national-data-science-bowl

National Data Science Bowl competition entry for the Best Whale Wow team of the Radboud University Nijmegen. We ended 68th.
1 stars 0 forks source link

Kmeans #14

Closed LucNies closed 9 years ago

LucNies commented 9 years ago

Added iterations to minibatch kmeans, move centroids image saver to util and fixed the problem of trying to write to non-existent directories.

StevenReitsma commented 9 years ago

You should only merge tested code into master really. Your second commit is probably okay but the first is not gonna work (e.g. this. instead of self.).

LucNies commented 9 years ago

I've fixed that mistake in the first commit and tested the last one, so there shouldn't be a problem

StevenReitsma commented 9 years ago

Well there's still the 'this.centroids' thing in the src/supervisedkmeans.py file.

LucNies commented 9 years ago

... i've could have sworn that I fixed that... Anyway, it really is fixed this time

Moorkopsoesje commented 9 years ago

I couldn't find any mistakes :)