madskjeldgaard / portedplugins

A collection of plugins for the SuperCollider sound environment, all of which are ported / remixed from elsewhere
GNU General Public License v3.0
180 stars 13 forks source link

more Tape plugins #34

Closed schollz closed 1 year ago

madskjeldgaard commented 1 year ago

This looks fantastic @schollz. Let me test it out and report back to you! Thanks so much..

madskjeldgaard commented 1 year ago

Tried them now @schollz. Absolutely stellar work. They work really nicely. Thanks a lot!

I just have a few questions for you to consider before we merge:

Thanks again for the great work!

schollz commented 1 year ago

Thanks @madskjeldgaard !

I completely forgot about the README! Yeah I was planning on fixing that up to.

I agree about better examples too... some of the fx are pretty subtle! I have way better examples from my own usage I can share.

Thanks for your link, I didn't know about your tutorial. It is astounding work. Re: smoothing, its a good idea. I removed it while debugging when something wasn't working and I erroneously thought it was the smoothing and then neglected to put it back!

Your feedback is greatly appreciated. I will follow up! Not sure when - maybe today, maybe next month... but please keep the PR open :)

madskjeldgaard commented 1 year ago

Thanks and again: excellent work !Sent from my iPhoneOn 26 Jan 2023, at 17.04, Zack @.***> wrote: Thanks @madskjeldgaard ! I completely forgot about the README! Yeah I was planning on fixing that up to. I agree about better examples too... some of the fx are pretty subtle! I have way better examples from my own usage I can share. Thanks for your link, I didn't know about your tutorial. It is astounding work. Re: smoothing, its a good idea. I removed it while debugging when something wasn't working and I erroneously thought it was the smoothing and then neglected to put it back! Your feedback is greatly appreciated. I will follow up! Not sure when - maybe today, maybe next month... but please keep the PR open :)

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

madskjeldgaard commented 1 year ago

Hey schollz - hope you are good. Any update on this? Really wanna get these wonderful plugins out!

schollz commented 1 year ago

Soon! I'm still thinking about it just haven't got a chance to sit at a computer long enough :)

schollz commented 1 year ago

I made it! Please have a look through - I added to the README and put better examples up.

I decided not to filter the parameters - I wasn't getting any issues without filtering and realized that some of the parameters in the UGens are recalculated on change so filtering would add a cost to the CPU (more steps of recalculation).

Happy to still make adjustments, won't be so long next time :)

madskjeldgaard commented 1 year ago

I made it! Please have a look through - I added to the README and put better examples up.

I decided not to filter the parameters - I wasn't getting any issues without filtering and realized that some of the parameters in the UGens are recalculated on change so filtering would add a cost to the CPU (more steps of recalculation).

Happy to still make adjustments, won't be so long next time :)

Looks excellent! I've used them for a bit now (and love them!!!) and haven't had any issues so it's probably fine. Thank you so much!

schollz commented 1 year ago

Whoa that was fast! Thanks :) I probably will be adding one more to your repo soon...

madskjeldgaard commented 1 year ago

Whoa that was fast! Thanks :) I probably will be adding one more to your repo soon...

You are always welcome. I should contribute a bit more myself soon as well, thanks for the inspiration!