chugins icon indicating copy to clipboard operation
chugins copied to clipboard

Rename Random chugin to Rand

Open heuermh opened this issue 3 years ago • 5 comments

See https://github.com/heuermh/lick/issues/11

Or alternatively, Rand could be called Gaussian or GaussianRandom.

heuermh avatar May 21 '21 16:05 heuermh

+1

Would it make sense to prefix this collection of chugins to avoid any collisions in the future?

benrogmans avatar May 22 '21 12:05 benrogmans

Would it make sense to prefix this collection of chugins to avoid any collisions in the future?

While we're at it, how about

Add namespaces and import statements https://github.com/ccrma/chuck/issues/109 😉

heuermh avatar May 24 '21 19:05 heuermh

Ping @gewang @spencersalazar @lathertonj for review

heuermh avatar Jul 20 '21 19:07 heuermh

Ping @nshaheed for review

heuermh avatar Jul 15 '22 18:07 heuermh

Im conflicted on this one -- Random is typically a 1st party-provided utility in many programming languages, and it seems like 3rd party libraries should use namespacing (whether actual namespaces or common prefixes) to avoid conflicts for things like this. Eventually I would think the Random chugin would expand to support more than just gaussian -- any assistance on improving it is also appreciated.

spencersalazar avatar Aug 24 '22 16:08 spencersalazar

I'm backing down on this one 😉

Refactored LiCK RandomChance in commit 7392215

heuermh avatar May 19 '23 15:05 heuermh