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

Migrate to teal data@78 simplify joinkeys@main #343

Closed
wants to merge 34 commits into from

Conversation

m7pr and others added 30 commits June 30, 2023 15:53
Merge branch 'main' of https://github.com/insightsengineering/teal.modules.hermes

# Conflicts:
#	R/km.R
#	man/tm_g_km.Rd
@m7pr m7pr added the core label Nov 16, 2023
Copy link
Contributor

badge

Code Coverage Summary

Filename              Stmts    Miss  Cover    Missing
------------------  -------  ------  -------  -------------------------------------------------------------------------------------------------------------------------------
R/adtteSpec.R           168     121  27.98%   248-393
R/assaySpec.R            53      39  26.42%   106-148
R/barplot.R             169     136  19.53%   38-62, 121-257
R/boxplot.R             172     144  16.28%   39-63, 117-258
R/checkmate.R            18       9  50.00%   110-118
R/experimentSpec.R       89      56  37.08%   97, 212-279
R/forestplot.R          196     171  12.76%   57-88, 144-306
R/geneSpec.R            256     153  40.23%   154-169, 296-480
R/km.R                  188     157  16.49%   60-89, 150-301
R/pca.R                 355     272  23.38%   33-53, 161-462
R/quality.R             298     228  23.49%   18-107, 201-429
R/sampleVarSpec.R       236     104  55.93%   293, 312-321, 327-334, 336, 344-356, 358-359, 361, 364, 372-376, 378-393, 398-422, 425-429, 431, 438-448, 450-451, 459, 504-521
R/scatterplot.R         167     137  17.96%   38-62, 120-256
R/utils.R                16       0  100.00%
R/volcanoplot.R         193     163  15.54%   33-54, 109-278
R/zzz.R                   1       1  0.00%    2
TOTAL                  2575    1891  26.56%

Diff against main

Filename           Stmts    Miss  Cover
---------------  -------  ------  -------
R/barplot.R           -2      -2  +0.23%
R/boxplot.R           -2      -2  +0.19%
R/forestplot.R        -3      -3  +0.19%
R/km.R                -3      -3  +0.26%
R/pca.R               -2      -2  +0.13%
R/quality.R           -2      -2  +0.16%
R/scatterplot.R       -2      -2  +0.21%
R/volcanoplot.R       -2      -2  +0.16%
TOTAL                -18     -18  +0.18%

Results for commit: 0bc4d93

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

Unit Tests Summary

    1 files    15 suites   12s ⏱️
  56 tests   43 ✔️ 13 💤 0
538 runs  525 ✔️ 13 💤 0

Results for commit 0bc4d93.

Copy link
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
adtteSpec 💔 $10.27$ $+1.60$ $+444$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
adtteSpec 💔 $0.35$ $+1.07$ h_km_mae_to_adtte_function_also_works_when_some_ID_variables_are_factors

Results for commit 0eab84d

♻️ This comment has been updated with latest results.

@m7pr m7pr closed this Nov 16, 2023
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.

Replace TealData with teal_data in the docs
1 participant