p5.js-website icon indicating copy to clipboard operation
p5.js-website copied to clipboard

p5.strands tutorial: rename the sphere() helper function

Open davepagurek opened this issue 6 months ago • 2 comments

Topic

When running the code in https://beta.p5js.org/tutorials/intro-to-p5-strands/, there's is a helper function called sphere here:

Image

This leads to some somewhat confusing code, as sphere() is already a thing in p5. Maybe we can rename this to something else?

davepagurek avatar Jun 18 '25 19:06 davepagurek

@davepagurek what about something like "randomPositionOnSphere"? Long but more descriptive and not in conflict

ksen0 avatar Jun 23 '25 16:06 ksen0

That sounds good to me!

davepagurek avatar Jun 28 '25 17:06 davepagurek

@davepagurek I'd like to work on this. Can you please assign it to me?

reshma045 avatar Jul 03 '25 07:07 reshma045

Thanks @reshma045!

davepagurek avatar Jul 03 '25 12:07 davepagurek

Closing as the associated PR has been merged. Thanks!

ksen0 avatar Jul 18 '25 16:07 ksen0