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

OceanVoyager-smwwims #466

Merged
merged 3 commits into from
May 29, 2024
Merged

OceanVoyager-smwwims #466

merged 3 commits into from
May 29, 2024

Conversation

smwwims
Copy link
Contributor

@smwwims smwwims commented May 13, 2024

Ocean Voyager from @smwwims
I've created this artwork containing a ship and a wave made of a sinus. You can change the amplitude, starting height and the resolution of the sinus. Apart from that, you can also change the size of the ship as well as the size and the orientation of the sun.

I have a bit of a coding experince in Python and Arduino C (mainly microcontroller coding). This is my first time coding in javascript and the first time I've completely read the documentation of a library.

0

Ocean Voyager from @smwwims
Copy link

vercel bot commented May 13, 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 May 18, 2024 4:50pm

@leomcelroy
Copy link
Contributor

This is a simple piece but it's nice. Maybe you could try clipping the bottom of the boat beneath the water.

My main suggestion before merging though is can you make the wave smoother? Or was it sort of blocky intentionally. I also think that waveResolution is actually referring to frequency. Resolution suggests the sampling size of the discretization of the sin function.

Smoothened sin wave
Updated snapshots
@smwwims
Copy link
Contributor Author

smwwims commented May 18, 2024

Thank you! I smoothened the wave through the catmullRom function and changed the variables for resolution and frequency.

0

@leomcelroy
Copy link
Contributor

Nice congrats on earning your Blot.

You can order it here.

@leomcelroy leomcelroy merged commit 7e089e0 into hackclub:main May 29, 2024
1 check passed
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.

None yet

2 participants