◐ Shell
reader mode source ↗
Skip to content

Add additional math methods#140

Open
tychedelia wants to merge 5 commits into
processing:mainfrom
tychedelia:feature/135-math-methods
Open

Add additional math methods#140
tychedelia wants to merge 5 commits into
processing:mainfrom
tychedelia:feature/135-math-methods

Conversation

@tychedelia

Copy link
Copy Markdown
Member

@tychedelia tychedelia changed the title Feature/135 math methods Apr 25, 2026
@SableRaf

SableRaf commented Apr 25, 2026

Copy link
Copy Markdown

Could we explicitly address the tradeoffs before making a decision on map() vs remap()?

From issue #131 it seems like there are at least two possible directions:

  1. Keep map() for Processing compatibility, possibly by dispatching based on arguments like Processing.py does.
  2. Use remap() to avoid shadowing Python’s built-in map(), like py5 does (see @hx2A's comment on 131)

Note: we should document this type of decisions on GitHub, since not everyone is on Discord and messages there are not publicly available without an account.

@SableRaf SableRaf self-requested a review April 25, 2026 06:18
@tychedelia

Copy link
Copy Markdown
Member Author

Could we explicitly address the tradeoffs before making a decision on map() vs remap()?

I feel pretty strongly that shadowing built-ins is the wrong decision. map is a pretty common functional programming idiom in python and it would be frustrating if users were left with the option either to not use the wildcard import (requiring manually importing every processing API item one-by-one) or not be able to express common python patterns.

@catilac

catilac commented May 3, 2026

Copy link
Copy Markdown
Contributor

Could we explicitly address the tradeoffs before making a decision on map() vs remap()?

I feel pretty strongly that shadowing built-ins is the wrong decision. map is a pretty common functional programming idiom in python and it would be frustrating if users were left with the option either to not use the wildcard import (requiring manually importing every processing API item one-by-one) or not be able to express common python patterns.

I want to second this. We don't want to clobber the python ecosystem in being overly opinionated. If this somehow becomes an issue to downstream users we can address it then.

is this ready?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add math methods sin(), cos(), tan(), abs() etc.

3 participants