[p5.js 2.0 Bug Report]: Not all primitive shape drawing goes through internal p5.Shape

[davepagurek]

Most appropriate sub-area of p5.js?

• Accessibility
• Color
• Core/Environment/Rendering
• Data
• DOM
• Events
• Image
• IO
• Math
• Typography
• Utilities
• WebGL
• Build process
• Unit testing
• Internationalization
• Friendly errors
• Other (specify if possible)

p5.js version

2.x

Web browser and version

All

Operating system

All

Steps to reproduce this

Not exactly a bug, but, in debugging #7903, we noticed that while some drawing functions end up going through our internal p5.Shape representation, others manually apply drawing commands like ctx.ellipse(centerX, centerY, radiusX, radiusY, 0, 0, 2 * Math.PI). This works for now, but:

• we have more code paths to consider when we need to make changes to rendering, and more to maintain
• in the future, if we want to be able to expose p5.Shape to users (e.g. via a future loadSVG() function), we won't have primitives ready to turn those shape components into

It would be great if we could add support for those primitives in p5.Shape, which involves:

• Making a subclass of ShapePrimitive for arcs and ellipses
• Updating the PrimitiveToPath2DVisitor to be able to convert those new primitives back into path drawing commands
• Also updating the PrimitiveToVerticesVisitor to be able to convert those to vertices in WebGL
• Updating the code that renders arcs and ellipses to now use p5.Shape instead of manually issuing drawing commands

[ayushman1210]

hi @davepagurek i would like to work on this issue please assign it to me

[ksen0]

Hi @ayushman1210, I've checked your currently open PRs, and looks like there are already outstanding reviews and questions for your to address on your open issues. So please work on these before taking on new work.

Thanks for making this @davepagurek! For now, I'll mark this issue as reserved for CodeDay.org - so only volunteers from the CodeDay internship will be assigned.

Thank you everyone!

[rounittxx]

hey @davepagurek @ksen0 i would like to work on this issue can you please assign it to me?

[rounittxx]

Hi @ksen0,

I realise this issue is reserved for CodeDay.org, but I’ve been studying the relevant parts of the codebase — specifically ShapePrimitive, the PrimitiveToPath2DVisitor, and PrimitiveToVerticesVisitor, as well as how arcs/ellipses currently bypass p5.Shape and call canvas APIs directly. I’m a beginner, but I’m comfortable exploring these internals and would like to take on implementing the missing primitive classes and updating the visitors/rendering paths.

If the reservation is strict, no problem — I’ll look for another issue. Otherwise, I’d appreciate the opportunity to work on this. Thanks!

[ksen0]

Hi @rounittxx ! Thanks for your interest. Since the next CodeDay round of work starts a bit later, we're happy to open this issue to you for work.

However, please create a PR by Dec 5th. If you're not finished by then, you can mark it as a "draft." So if it turns out you don't have time for it or if it needs a lot of additional work or iteration, we could reassign it. I hope that makes sense! (Typically, tasks do not have a specific time constraint, so you're welcome to look for other open work instead if Dec 5th is too soon.)

Please don't hesitate to ask for help form me or @davepagurek here or on Discord!

[rounittxx]

Yaa sure i m working on it.

[ksen0]

As an update: this issue is not currently being worked on, but it is reserved for a CodeDay microinternship. Thank you for understanding!

[sea-brenn]

I'm working on this.

[ksen0]

Hi @sea-brenn , Could you clarify please whether you're working on this via the CodeDay internship? Thanks so much!

[sea-brenn]

hi @ksen0 yes, working on it through the CodeDay internship.

[VANSH3104]

Hello @ksen0,
As you can see in the earlier discussion, I have already worked extensively on the alternative approach to this problem in PR #8236, which has now been merged. This work addressed the transform-related issues encountered while debugging #7903 and covered similar internal rendering paths.
Given this background and considering that my other assigned issues are already completed, I would like to inquire whether it would be acceptable for me to also contribute to this issue or if you would prefer it to remain exclusively within the CodeDay scope.
I am happy with either outcome; I simply wanted to ask before proceeding. Thank you for your time and guidance.

[ksen0]

Hi @VANSH3104 ! Thanks so much for all your work, as well as your clear question. For now, yes, the issue was reserved for the CodeDay collaboration so it's being worked on through that. If you want to contribute here, actually it would be very helpful if you were willing to review the work? We have been maintaining a list of area steward and if you're interested in stewardship (here's what it means for p5.js), then you'd be welcome to be involved that way? Also no pressure at all, but pointing out a possibility!