Skip to content

Conversation

Polkas
Copy link
Contributor

@Polkas Polkas commented Aug 30, 2022

closes #451

Fixed the join type functionality in the tm_t_crosstable.

@Polkas Polkas added the core label Aug 30, 2022
@Polkas Polkas mentioned this pull request Aug 30, 2022
15 tasks
@github-actions
Copy link
Contributor

badge

Code Coverage Summary

Filename                      Stmts    Miss  Cover    Missing
--------------------------  -------  ------  -------  ----------------------------------------------
R/tm_a_pca.R                    808     808  0.00%    70-1012
R/tm_a_regression.R             719     719  0.00%    91-902
R/tm_data_table.R               180     180  0.00%    150-394
R/tm_file_viewer.R              172     172  0.00%    38-240
R/tm_front_page.R               123     112  8.94%    61-199
R/tm_g_association.R            287     287  0.00%    88-430
R/tm_g_bivariate.R              616     456  25.97%   105-642, 690, 696, 700, 811, 828, 846, 866-888
R/tm_g_distribution.R           974     974  0.00%    107-1235
R/tm_g_response.R               323     323  0.00%    84-470
R/tm_g_scatterplot.R            685     685  0.00%    141-911
R/tm_g_scatterplotmatrix.R      260     242  6.92%    79-354, 407, 421
R/tm_missing_data.R            1038    1038  0.00%    56-1251
R/tm_outliers.R                 989     989  0.00%    71-1195
R/tm_t_crosstable.R             246     246  0.00%    80-368
R/tm_variable_browser.R         754     749  0.66%    46-1195
R/utils.R                        96      96  0.00%    74-290
R/zzz.R                           1       1  0.00%    2
TOTAL                          8271    8077  2.35%

Diff against main

Filename    Stmts    Miss    Cover
----------  -------  ------  -------
TOTAL       -        -       -

Results for commit: 27d2262

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions
Copy link
Contributor

Unit Tests Summary

  1 files    5 suites   0s ⏱️
16 tests 16 ✔️ 0 💤 0
49 runs  49 ✔️ 0 💤 0

Results for commit 04da5e6.

Copy link
Contributor

@gogonzo gogonzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Polkas Polkas merged commit 25ab061 into main Sep 2, 2022
@Polkas Polkas deleted the 451_cross_merge@main branch September 2, 2022 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cross table - merge_fun -> join_fun

2 participants