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

flowered-ekqiu #989

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

flowered-ekqiu #989

wants to merge 12 commits into from

Conversation

ekqiu
Copy link

@ekqiu ekqiu commented Nov 6, 2024

Initially i wanted to create a canvas of stars, but to make it more unique and generative, I had added constant offset points to the lines to create a sort of curved flower. It ranges from thick to thin petals, and has varying length.

  • I have read the steps to getting a blot
  • I have read all the submission requirements and follow them
  • I am submitting art that...
    • is algorithmically generated (will meaningfully change each time the program is run)
    • is your own work that is not copied from somewhere else
    • is drawable through the editor(does not use outside tools)
    • doesn't call Math.random() (See the documentation on randomness)
    • is drawable on Blot itself (doesn't have lines overlap more than 5 times)
      • Doesn't overlap lines more than 5 times
      • Fits in the working area of 125x125mm
      • Doesn't rely on colour to look good

If you used based your art on something else, used a tutorial, or remixed it from something else, please link it here:

Lastly, link a picture of your art piece here:

Copy link

vercel bot commented Nov 6, 2024

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

A member of the Team first needs to authorize it.

Copy link

github-actions bot commented Nov 6, 2024

art/flowered-ekqiu/index.js looks like art! preview it in the editor

@ekqiu ekqiu changed the title Add files via upload flowered-ekqiu Nov 6, 2024
@EnterpriseGoose
Copy link
Contributor

Hello! Sorry it's been so long. The main reviewer has been swamped lately, so I wanted to give you a few hints so you have the best chance of getting a blot.

Overall I think you have an interesting piece of generative art here. The shapes that can generate are really cool and the way it sometimes gets a 3d effect is awesome. To get a blot, though, I think you'll have to make a few changes.

The biggest issue is the overlapping lines. I noticed you didn't check these boxes in the pr checklist, so I appreciate you being careful with looking over that and being honest, but this amount of overlap will unfortunately make it undrawable. If you were to try, it would definitely rip the paper, especially in the center.
image
Is there some way you can make this in a way that avoids doing too much overlap, especially avoiding the center getting too full?

Other than that, I would just want to see a little more complexity and artistry. Generally, people spend ~15-20hrs on a "blot-worthy" project.

I look forward to seeing where you go with this! Just update your fork (the pr will automatically update to your fork) and send a message here when you're ready to have me take another look.

@ekqiu
Copy link
Author

ekqiu commented Nov 18, 2024

Hey @EnterpriseGoose, thanks for your review. Understand that there's too much overlap, and I have heeded your advice and made some adjustments. The lines are now less dense in the middle, and I was able to get some snapshots of some different flowers. Not sure why the lines one the images are thicker than the ones on the Blot IDE. Hope that you'll approve of this. Thanks!

@EnterpriseGoose
Copy link
Contributor

It's looking great! It definitely seems more drawable now. I would still love to see if you can add a bit more complexity. Maybe you can figure out another element or two, or find some way to make the flower even more flower-y. I just would love to see the project be a bit more polished and feel more like a finished piece of art. I think you can do it!

@ekqiu
Copy link
Author

ekqiu commented Jan 8, 2025

Hi! sorry for the long inactivity, was on holiday and completely forgot to make the edits on this. i hope its much better now, as it is more representative of a "flower".

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.

2 participants