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

DEV #68

Merged
merged 179 commits into from
Jul 10, 2024
Merged

DEV #68

merged 179 commits into from
Jul 10, 2024

Conversation

randrescastaneda
Copy link
Owner

Joyn 0.2.2

  • Remove unmasking functions as we realized that they are not necessary.

joyn 0.2.1 (Not in CRAN)

  • Add anti_join() function.

  • Add unmask_joyn() function to unmask joyn functions that mask dplyr equivalents.

  • Add information about duplicated obs in by variable when match type is 1 rathern than m.

  • improve inefficiencies in deep copies with m:m joins

  • Replace m:m joins from data.table::merge.data.table to collapse::join. Thanks to @SebKrantz for the suggestion (Just a question regarding usage of collapse::join #58).

  • Add information about duplicated obs in by variable when match type is 1 rather than m.

  • Internal: improve storing of joyn messages.

  • Improve creation of reporting variable. Now, it is created in [collapse::join] rather than in joyn function. In addition, the reporting variable is created as factor to improve performance. Thanks to @SebKrantz for the suggestion (Just a question regarding usage of collapse::join #58)

breaking changes

  • Now, by default, joyn will not sort the data. This is to avoid unnecessary
    computational time that most of the time is not needed.
    If the user wants to sort the data, they can use the sort argument, which triggers
    the sorting mechanism of collapse package.

  • report variable (named ".join" by default) is now a factor instead of character. Yet, users can still use character if they want with the reporttype = "character".

@randrescastaneda randrescastaneda merged commit 311460d into master Jul 10, 2024
8 checks passed
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.

3 participants