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

Adding PAM implementation #12

Closed
wants to merge 2 commits into from
Closed

Adding PAM implementation #12

wants to merge 2 commits into from

Conversation

lendle
Copy link
Contributor

@lendle lendle commented Feb 5, 2014

Example usage in combination with the Distance package: http://nbviewer.ipython.org/gist/lendle/8820029

@johnmyleswhite
Copy link
Member

Very cool. What's the license for the materials you've based this on?

@lendle
Copy link
Contributor Author

lendle commented Feb 5, 2014

Good question. One reference looks like class notes (http://www.cs.umb.edu/cs738/pam1.pdf), and I do not see any license information on what appears to be the course website. The other two from wikipedia and wikibooks are have the Creative Commons Share Alike license.
I only used the descriptions of the algorithms in the references, not any code.

@johnmyleswhite
Copy link
Member

If you only looked at descriptions, we're totally fine. It's only code that's troubling, since you can't copyright algorithms, only code.

@lendle
Copy link
Contributor Author

lendle commented Feb 5, 2014

Okay sounds good. I cited the first one in particular so it would be clear where my notation came from and because that formulation of the algorithm is not very intuitive.

By the way, are there plans at this point for a coherent clustering result type? I think that would make sense at last for partitioning methods.

@johnmyleswhite
Copy link
Member

There aren't plans, but we'd be pretty open to hearing what you think we should do. Make things more generic would be a big gain for us.

@lendle
Copy link
Contributor Author

lendle commented Feb 5, 2014

I moved this comment to #13

@lindahua
Copy link
Contributor

lindahua commented Feb 5, 2014

@lendle: may I suggest that you open a new issue specifically to discuss generic clustering interface?

It is better that this issue is for the PAM method.

@lendle
Copy link
Contributor Author

lendle commented Feb 7, 2014

I'm canceling this pull request so I can work on the PAM implementation a bit more, and because the package is getting moved to JuliaStats. I'll resubmit when I have a more mature implementation.

@lendle lendle closed this Feb 7, 2014
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.

3 participants