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

added KMeans_mouseClustering to examples page #1282

Merged
merged 1 commit into from
Jan 28, 2022

Conversation

tlsaeger
Copy link
Member

Dear ml5 community,

I'm making a Pull Request(PR). Please see the details below.

A good PR 🌟

β†’ Step 1: Describe your Pull Request πŸ“

Fixing a Bug? Adding an Update? Submitting a New Feature? Does it introduce a breaking change?

Linking the new example for kmeans (discussed here #1146 ) to the docs.
Both a link the to the examples folder in the repo as well as to the p5 web editor.
Thanks @bomanimc for creating the web editor link.

A great PR 🌟🌟

β†’ Step 2: Share a Relevant Example πŸ¦„

Here's some example code or a demonstration of my feature as a part of this pull request, a separate pull request, in the https://editor.p5js.org, or codepen/jsfiddle/etc...

The best PR 🌟🌟🌟

β†’ Step 3: Screenshots or Relevant Documentation πŸ–Ό

Here's some helpful screenshots and/or documentation of the new feature

@tlsaeger tlsaeger added documentation kmeans SEMVER/patch a version tag for a patch release change labels Jan 22, 2022
@joeyklee joeyklee merged commit 9245611 into main Jan 28, 2022
@joeyklee joeyklee deleted the tlsaeger-mouseClustering-added branch January 28, 2022 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation kmeans SEMVER/patch a version tag for a patch release change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants