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

Workflow propagation #217

Merged
merged 1 commit into from
Dec 12, 2023
Merged

Workflow propagation #217

merged 1 commit into from
Dec 12, 2023

Conversation

@walkowif walkowif requested a review from a team December 11, 2023 14:19
Copy link
Contributor

badge

Code Coverage Summary

Filename                   Stmts    Miss  Cover    Missing
-----------------------  -------  ------  -------  ------------------------
R/assertthat.R                19       1  94.74%   149
R/calc_cor.R                  25       0  100.00%
R/checkmate.R                  4       0  100.00%
R/connections.R              143     125  12.59%   27-161, 215-287, 356-381
R/differential.R             101       3  97.03%   190, 193, 196
R/dplyr_compatibility.R        3       1  66.67%   23
R/draw_barplot.R              34       0  100.00%
R/draw_boxplot.R              71       5  92.96%   81, 88-89, 106, 109
R/draw_heatmap.R              35       1  97.14%   71
R/draw_scatterplot.R          32       4  87.50%   73-74, 87, 93
R/GeneSpec-class.R            68       0  100.00%
R/graphs.R                   107       3  97.20%   107-109
R/HermesData-class.R          35       4  88.57%   134-137
R/HermesData-methods.R       194       0  100.00%
R/HermesData-validate.R       44       0  100.00%
R/join_cdisc.R                37       0  100.00%
R/normalization.R            100       2  98.00%   194, 223
R/pca_cor_samplevar.R         59       0  100.00%
R/pca.R                       28       0  100.00%
R/quality.R                   75       3  96.00%   109-112
R/top_genes.R                 51       0  100.00%
R/utils.R                     98       0  100.00%
TOTAL                       1363     152  88.85%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: e069cb5

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

Unit Tests Summary

    1 files    23 suites   1m 4s ⏱️
273 tests 256 ✔️ 17 💤 0
849 runs  827 ✔️ 22 💤 0

Results for commit e069cb5.

Copy link
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
HermesData-methods 💚 $9.85$ $-4.21$ $0$ $0$ $0$ $0$
calc_cor 💚 $5.02$ $-2.25$ $0$ $0$ $0$ $0$
differential 💚 $52.08$ $-23.24$ $0$ $0$ $0$ $0$
graphs 💚 $4.50$ $-2.21$ $0$ $0$ $0$ $0$
normalization 💚 $18.30$ $-6.64$ $0$ $0$ $0$ $0$
pca 💚 $7.30$ $-3.14$ $0$ $0$ $0$ $0$
top_genes 💚 $5.82$ $-2.46$ $0$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
HermesData-methods 💚 $5.63$ $-2.94$ lapply_works_as_expected_for_an_MAE
calc_cor 💚 $3.55$ $-1.61$ autoplot_for_HermesDataCor_works_as_expected_with_custom_options
differential 💚 $9.35$ $-4.29$ diff_expression_allows_passing_of_method_arguments_to_helper_functions
differential 💚 $10.18$ $-5.57$ diff_expression_works_as_expected_with_the_DESeq2_method
differential 💚 $16.74$ $-7.24$ h_diff_expr_deseq2_can_pass_arguments_to_DESeq2_DESeq
differential 💚 $9.18$ $-4.54$ h_diff_expr_deseq2_works_as_expected
normalization 💚 $6.32$ $-2.13$ h_rlog_function_works_as_expected_with_custom_settings
normalization 💚 $5.30$ $-1.35$ h_rlog_function_works_as_expected_with_default_settings

Results for commit 97d6537

♻️ This comment has been updated with latest results.

@danielinteractive
Copy link
Collaborator

Note that BiocCheck result can be ignored.

@walkowif walkowif merged commit a97a7f5 into main Dec 12, 2023
22 of 23 checks passed
@walkowif walkowif deleted the propagation-december-2023 branch December 12, 2023 09:03
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.

2 participants