-
Notifications
You must be signed in to change notification settings - Fork 34
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
overhaul format conversion functions #53
Comments
Because the |
There is now a whole book on tidy evaluation, which may be useful for improving the readability and efficiency of the parameter change. |
Bump. Also, as the established |
I understand. I could take a look. There are some useful docs on the rlang website. I think it is worth removing the dep on lazyeval to make things more consistent. I also spotted a defunct function: |
The convenience functions
is_*_form()
andto_*_form()
need a few upgrades:weight
parameter, whereas the layers have switched toy
, which may cause confusion.to_*_form()
rely ontidyr::gather()
andtidyr::spread()
, which are stable but outdated.to_lodes_form()
cannot handle alluvial data with variabley
(andweight
) values. The new functiontidyr::pivot_longer()
makes this possible, as illustrated in Longitudianal data #51. It should be possible to similarly convert from lodes to alluvia form while preservingy
(andweight
) values usingtidyr::pivot_wider()
.For consistency between these functions and the layers, it may be prudent to deprecate
axis[0-9]+
in favor ofx[0-9]+
, which would better harmonize with the naturaly[0-9]+
that would be used to specify variabley
values.The text was updated successfully, but these errors were encountered: