Skip to content

demo: Add a demo#118

Merged
gaocegege merged 3 commits intomasterfrom
example/demo/figure
Jul 17, 2017
Merged

demo: Add a demo#118
gaocegege merged 3 commits intomasterfrom
example/demo/figure

Conversation

@gaocegege
Copy link
Copy Markdown
Member

@gaocegege gaocegege commented Jun 13, 2017

ref https://www.openprocessing.org/sketch/377464

Just for fun

Delay until #97
Signed-off-by: Ce Gao ce.gao@outlook.com

Signed-off-by: Ce Gao <ce.gao@outlook.com>
@jeremydouglass
Copy link
Copy Markdown
Member

jeremydouglass commented Jun 13, 2017

Tested this, and it ran fine. Nice adaptation of the reference sketch -- your original wobbly ellipse phase is particularly fun. :)

Remind me: why is:

radians = RLangPApplet$radians

...something that must be defined in src/rprocessing/r/core.R -- why can it not simply be added to the top of FiguresTransformation.rpde as a local definition? I know that it does not work, but what is the explanation?

@gaocegege
Copy link
Copy Markdown
Member Author

I think it is caused by our policy to eval the code, but I am not sure the specific reason. 🤔

@jeremydouglass
Copy link
Copy Markdown
Member

Could this be resolved now that #97 has a functional workaround?

@gaocegege
Copy link
Copy Markdown
Member Author

Yeah, I will refactor the code and merge it.

@gaocegege gaocegege merged commit 39919eb into master Jul 17, 2017
@gaocegege gaocegege deleted the example/demo/figure branch July 17, 2017 02:50
@gaocegege gaocegege added this to the Evaluation 2 milestone Jul 17, 2017
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.

2 participants