Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

BeautifulSaturn-AlexB #606

Merged
merged 16 commits into from
Jul 15, 2024
Merged

BeautifulSaturn-AlexB #606

merged 16 commits into from
Jul 15, 2024

Conversation

oldfashionedjelly
Copy link
Contributor

This was a lot harder than I expected, but I'm happy with how it turned out. It took a lot longer than I expected and it pushed me out of my comfort zone with using JavaScript and Blot. However, I'm happy that I put in all of the time that I did. There was an immense amount of trial and error required as I went through at least five different design ideas, but I chose this one because it is simple yet artistic and can be made pretty easily with the machine. It features the planet Saturn with some stars in the background. You can also change things like the size and angle of the planet and rings and the number and size of the stars.

snapshot1 snapshot2

Copy link

vercel bot commented Jun 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
blot-3dgallery ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 13, 2024 1:55pm

@leomcelroy
Copy link
Contributor

This is almost there!

Do you think you could have the back part of the ring covered by the planet?

Also make sure the snapshots folder is all lowercase.

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.

ooooh, out of this world! Yeah, if you fix the things that @leomcelroy mentioned I can merge this

@oldfashionedjelly
Copy link
Contributor Author

snapshot3

I finished making all of the changes!

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.

Nice work! A couple other things come to mind though– the stars show through the rings. Also, in most photos of saturn, it looks like there are a couple rings. What if you add that in?

Screenshot 2024-07-14 at 03 18 17

@oldfashionedjelly
Copy link
Contributor Author

snapshot1

How's this?

@maxwofford
Copy link
Member

Lol, that's one way to do it

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.

It's looking really nice now! Almost ready for merge. One last thing: it's not quite generative enough– I tried rendering it a couple times and only the stars move their positions and counts. A couple things you could try out:

  • variable number of rings each render
  • variable ring size
  • variable ring rotation (oooh, this would look dope)
  • different star size (smaller stars could look further away & form constellations)
  • different planet size

Once you have a couple of those things in this art deserves a blot!

@oldfashionedjelly
Copy link
Contributor Author

I fixed the star placement, scaled the planet and ring size by a random factor each time, made the ring rotation vary, and made different star sizes.

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 looks fantastic! Unfortunately it's still running outside the (blue) bounding box. Fix that so I can merge!

@oldfashionedjelly
Copy link
Contributor Author

It should be fixed now!

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.

Absolutely beautiful work on this! It def deserves a blot. Go ahead and order yours now!

@maxwofford maxwofford merged commit 908c024 into hackclub:main Jul 15, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants