-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support more built-in functions #16
Comments
Gaocegege, I am interested in contributing to this repo, If you have any direction of what issues, or what built in functions you would like to prioritize I would be happy to claim some of the work and send pull requests. |
@PatrickMurphy Thanks for your interest 🎉 Today, I'll write some documentation about the architecture of Processing.R and tag some issues to "for new contributors". If you have any problems when you read the code, feel free to @ me in GitHub or https://gitter.im/gaocegege/Processing.R. |
ref #16 (comment) Processing.R will focus on 2D & 3D Primitives at the first. Functions such as arc() and ellipse() are easy to implement. If you want to contribute, this is a good entrypoint :) |
@gaocegege -- this built-in functions list includes Is it part of the plan support mouse input and interaction like |
Also, should |
box in Processing has two definitions: |
Hi, there are too many built-in functions in Processing, so I am not sure if I have the ability to implement all functions on my own in the first evaluation. I am trying to attract some contributors to Processing.R. We could also talk about the priority between the functions and have a more detailed plan. |
@gaocegege -- You definitely should not try to implement all the built-ins on your own! Your plan to call for contributors is a good one. I would suggest either removing Strange, I thought you had implemented both versions of box: |
Oh, I forgot that. 🤔 Thanks for your review. |
Yeah, I will remove cursor from the list. |
Now renjin supports float correctly, so no need to do this work manually. |
Fastastic! Great ask, hooray for the Renjin core developers, and hooray for open source. ...and that should simplify hundreds of |
Now I think we could close the issue 🎉 |
So at this point every method from the API is automatically inherited? And the only remaining issue is with variables -- width, height, frameCount, frameRate, looping, focused, etc? |
Yes, I think so 🤔 |
Since this is closed, is there a separate, open issue for variables -- Perhaps edit the issue and remove them from the list, and add them to a separate issue for "Built-in variables" (vs. built-in functions). |
Similarly, "Creating & Reading" is listed in the issue as "Blocked until #9" -- if this is still true then create another separate issue for Creating and Reading built-ins and delete them from this closed issue built-ins list. |
Let's clean up this old list. Can all the "blocked until 9" boxes be checked off now that #86 is closed? |
All the functions in the list are supported. |
Now Processing.R supports draw, setting and some other functions but not enough.
ref https://processing.org/reference/
2D Primitives
Curves
Environments
3D Primitives
Color
Setting
Creating & Reading
Blocked until #9
The text was updated successfully, but these errors were encountered: