This PR is only to discuss our response to reviewers about the use of eval(parse()).
The answer I suggest it here. You may edit the corresponding ".Rmd" document and re-render. Once we are happy you may copy the raw text of the ".md" file and paste it in the review thread. Before that please see and remove my NOTES IN CAPS.
tl;dr
I would:
stick to eval(parse()).
ensure we test all equations are valid (the ones provided both by us and the users).
The reviewer jeffreyhanson seems to support these points.
Don't merge.
This PR is only to discuss our response to reviewers about the use of
eval(parse())
.The answer I suggest it here. You may edit the corresponding ".Rmd" document and re-render. Once we are happy you may copy the raw text of the ".md" file and paste it in the review thread. Before that please see and remove my NOTES IN CAPS.
tl;dr
I would:
eval(parse())
.The reviewer
jeffreyhanson
seems to support these points.