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

update #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

update #2

wants to merge 1 commit into from

Conversation

ameliapqy
Copy link

No description provided.

@rahwang
Copy link
Contributor

rahwang commented Sep 16, 2021

Nice work on the first two! Notice that the pulsing flower is a little slower at the beginning and spikes out faster at the end -- you can use a bias function, or square / cube the input to the amplitude scaler in the future. But lovely work, you clearly got the ideas :)

Could you please make the new flower public or unlisted? I think I don't have permission to view it.

@ameliapqy
Copy link
Author

It's changed! Thanks for the feedbacks :)

@rahwang
Copy link
Contributor

rahwang commented Sep 16, 2021

Thanks! And yoooo! It looks awesome. Nice work :D

ndevlin added a commit to ndevlin/lab01-toolbox-functions that referenced this pull request Sep 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants