Skip to content
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

call_standardise handles ~ incorrectly #473

Closed
garrettgman opened this issue Mar 29, 2018 · 1 comment
Closed

call_standardise handles ~ incorrectly #473

garrettgman opened this issue Mar 29, 2018 · 1 comment
Milestone

Comments

@garrettgman
Copy link

@garrettgman garrettgman commented Mar 29, 2018

I think this output is unintended

rlang::call_standardise(Sepal.Length ~ Sepal.Width)
## Sepal.Length ~ (Sepal.Length ~ Sepal.Width)

It creates an infinite recursion error in my code.

@garrettgman
Copy link
Author

@garrettgman garrettgman commented Mar 30, 2018

pryr's version seems to do something more sensible:

pryr::standardise_call(Sepal.Length ~ Sepal.Width)
## Sepal.Length ~ Sepal.Width
@lionel- lionel- added this to the 0.3.0 milestone May 12, 2018
@lionel- lionel- closed this in dcff091 Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants