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

Submission for lab (Nathaniel Korzekwa) #1

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

Conversation

ciscprocess
Copy link

No description provided.

@rahwang
Copy link
Contributor

rahwang commented Sep 16, 2021

Nice work! This looks very close to the original, but a little bit cooler since you went with the cubic pulse instead of the bias function I used. ;)

Definitely looks like you two understand toolbox functions!

@ciscprocess
Copy link
Author

Nice work! This looks very close to the original, but a little bit cooler since you went with the cubic pulse instead of the bias function I used. ;)

Definitely looks like you two understand toolbox functions!

Thanks! I am a bit sad I didn't get to the other 2 shaders, but I'm still a bit of a noob when it comes to parameter tuning. I still feel a bit like I'm playing pin the tail on the donkey lol.

Additionally, one reason developing shaders takes me so long (and a reason why 460 was a big workload) is it's very hard to test individual modules functions/modules in the code. I know we can set colors, etc, but that is pretty limited in how helpful it is. Do you know of any good shader testing environments or strategies, or is off debugging just a way of life?

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