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

aFractalForayIntoTheForest-Adya #762

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

Conversation

AdyaS2010
Copy link

@AdyaS2010 AdyaS2010 commented Jul 29, 2024

  • I have read the steps to getting a blot, had fun programming the artwork (I had to take a little time to adjust to the environment though).

  • 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)
    • calls bt.randInRange
  • I did get inspiration (of the idea to create a tree made of fractals) in the real world.
    FractalTree -> This image is not correct, check files for latest please.

  • [x]

@AdyaS2010 AdyaS2010 changed the title aFractalForayIntoTheForestAdd files via upload aFractalForayIntoTheForest - Adya Jul 31, 2024
@AdyaS2010 AdyaS2010 changed the title aFractalForayIntoTheForest - Adya aFractalForayIntoTheForest-Adya Jul 31, 2024
@BrightTheBackpack
Copy link
Member

Hey there! Make sure all your snapshots are in a /spapshots/ folder, and rename your js file to index.js! Also make sure aren't using Math.random()

Made a couple changes to avoid using Math.random()
Copy link

github-actions bot commented Aug 1, 2024

art/FractalTree-Adya/index.js looks like art! preview it in the editor

@AdyaS2010
Copy link
Author

Hey there! Make sure all your snapshots are in a /spapshots/ folder, and rename your js file to index.js! Also make sure aren't using Math.random()

Got it. Done!
Could you please check again?

@BrightTheBackpack
Copy link
Member

Now it doesn't seem to be different on every run. you can use bt.randInRange instead of math.random

Updated code to incorporate bt.randInRange to inculcate difference in output due to variable change! 

This results in a slight difference in the branches every time!
@AdyaS2010
Copy link
Author

Ok, done!

Copy link
Member

@maxwofford maxwofford 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 good start @AdyaS2010, but I think you can go further! Keep in mind most accepted submissions take 15-20 hours of work.

@AdyaS2010
Copy link
Author

AdyaS2010 commented Aug 7, 2024

This is a good start @AdyaS2010, but I think you can go further! Keep in mind most accepted submissions take 15-20 hours of work.

Should I expand on top of the existing model, experiment with new ideas, and branch out from there, or restart the whole thing?

Attempted to add leaves to the branches - Not Done Yet
Lol, this happened.
Basics for tree ...
@AdyaS2010
Copy link
Author

@BrightTheBackpack @maxwofford (sry for the ping) I have updated my Project after spending a lot more time on it; had a lot of fun, learnt so much, and am proud of my work! Thank you guys so much for putting this together and providing such opportunities!

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.

Hey! This art piece is great, but a lot of it is static.Could you add some more generative/varied elements?

Copy link

vercel bot commented Sep 6, 2024

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

A member of the Team first needs to authorize it.

@BrightTheBackpack
Copy link
Member

Also make sure all your files are in a folder with your art name, and your name!

@AdyaS2010
Copy link
Author

Also make sure all your files are in a folder with your art name, and your name!

Done!!! Is the artwork piece all good now?

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.

4 participants