-
Notifications
You must be signed in to change notification settings - Fork 51
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
Grackle: port stdlib #1727
Comments
Note to self:
|
NOTE: I must call toRadians on the Grackle backend for all trig functions... |
The reason why I have those functions as radians is I could not find a popular programming language where they were not radians, meaning any ai model and human is going to assume they are radians, if we are the outlier its unintuitive |
Oh so we want cos and sin, etc to remain radians? Because that's 100% what I want lol, but I know we have the whole "default to degrees" thing. That's perfect then. I'll just do it in the functions that take an |
These functions are the exception since it’s unintuitive |
Make them work like the current stdlib which is radians |
|
We are only porting the current stdlib adding new functions can be outside this |
The current executor supports around 66 stdlib calls. Currently, only
startSketchAt
,extrude
,close
andlineTo
have been ported to Grackle. Port the rest.The text was updated successfully, but these errors were encountered: