Dave Pagurek

Results 715 comments of Dave Pagurek

yeah I think if we rename this one we'd probably also want to rename the others to have the same naming convention, `defaultColorShader()` and `defaultMaterialShader()` maybe? Or to be less...

btw @Garima3110 feel free to ignore the merge conflicts with the dev-2.0 branch for now. If things are working on this branch, once the project is done I can handle...

Agreed, filters shouldn't be affected here. If you're curious, I was talking a bit here https://github.com/processing/p5.js/pull/7270#issuecomment-2409005444 about refactoring how some of these things work so that there are "safe" functions...

One other thing to mention: @perminder-17 was looking into replacing libtess with earcut here https://github.com/processing/p5.js/pull/7492, which is on pause because other changes were higher priority, but we intend to get...

I think this is happening because while regular meshes [take in a per-vertex color in their shader](https://github.com/processing/p5.js/blob/d51184b5598e4d51dca12905aeb3387b2e6a95bf/src/webgl/shaders/normal.vert#L4) and [pass it along](https://github.com/processing/p5.js/blob/d51184b5598e4d51dca12905aeb3387b2e6a95bf/src/webgl/shaders/normal.vert#L40), [the point shader does not](https://github.com/processing/p5.js/blob/dev-2.0/src/webgl/shaders/point.vert#L1). Updating the point shader...

Btw there's an issue here https://github.com/processing/p5.js-website/issues/636 you can follow for the old methods still showing up on the site!

In case it's helpful, in the p5.js Discord, you can make a new group in the Groups section (some local meetups use that!) So feel free to do whatever's easiest,...

I think the relative path from `/` technically will work once it gets merged and then parsed in the website repo, but I think as @perminder-17 suggests, an absolute path...

Hi @Dhanush111! English docs actually come from the p5.js repo, so any changes made directly here will get overwritten in the next release. Could you make your changes there instead?...

Thanks, looks good! > I understand why this function doesn't support string based colors Do you mean this as in, we don't *want* to support this, or the `color()` function...