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

HeartOfCircles #823

Closed
wants to merge 8 commits into from
Closed

HeartOfCircles #823

wants to merge 8 commits into from

Conversation

PawiX25
Copy link

@PawiX25 PawiX25 commented Aug 13, 2024

  • I have read the steps to getting a blot

  • I am submitting art that...

    • is algorithmically generated (will change each time the program is run)
    • is drawable on a blot (fits in the work area & doesn't overlap too much)
    • is original (not copied from somewhere else)
    • doesn't call Math.random() (See the documentation on randomness)
    • is drawable on a physical machine (doesn't have lines overlap more than 5 times)
  • Optional, if you used a tutorial or based your art on something else, please include the link here:

  • Optional, if you remixed this from something else, mention it here:

Copy link

github-actions bot commented Aug 13, 2024

art/HeartOfCircles-PawiX25/index.js looks like art! preview it in the editor

@PawiX25 PawiX25 changed the title Add files via upload HeartOfCircles Aug 18, 2024
Copy link
Member

@qcoral qcoral left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a really cool piece, and it's pretty close to being merged! I just ask that the scene has a bit more detail. I'll come back later!

Copy link

vercel bot commented Aug 27, 2024

@PawiX25 is attempting to deploy a commit to the Hack Club Team on Vercel.

A member of the Team first needs to authorize it.

@PawiX25
Copy link
Author

PawiX25 commented Aug 27, 2024

Thank you for the feedback! I’ve added more detail to the scene by introducing a background pattern with varied shapes, including circles, ellipses, and polygons. Looking forward to your thoughts on these updates!

@PawiX25
Copy link
Author

PawiX25 commented Sep 14, 2024

Hey @qcoral! I’ve made the changes you requested, including heart detection, transformations, and tweaks to avoid overlap. I also added some extra elements like circles, ellipses, and spirals. Let me know if there's anything else! Thanks for your feedback—it really shaped the piece!

@Dongathan-Jong
Copy link
Collaborator

Neat update! Just a small flaw, circles will generate outside of the border, and the blot cannot draw that. Once you fix that, please create a new PR once you have updated your art! We are doing this due to new management of the PR system.

@PawiX25
Copy link
Author

PawiX25 commented Oct 6, 2024

I've fixed it here: #946

Let me know if you have any other questions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants