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

Fix reporting issues #285

Merged
merged 3 commits into from
Oct 7, 2022
Merged

Fix reporting issues #285

merged 3 commits into from
Oct 7, 2022

Conversation

nikolas-burkoff
Copy link
Contributor

Closes #284

image

and fixed title of volcano plot

@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2022

Unit Tests Summary

  1 files  15 suites   23s ⏱️
58 tests 36 ✔️ 22 💤 0
85 runs  63 ✔️ 22 💤 0

Results for commit 39f08f5.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2022

badge

Code Coverage Summary

Filename              Stmts    Miss  Cover    Missing
------------------  -------  ------  -------  --------------------------------------------------------------------------------------------------------------------------
R/adtteSpec.R           157      11  92.99%   308-312, 358-362, 364
R/assaySpec.R            44      36  18.18%   102-142
R/barplot.R             162     129  20.37%   40-64, 122-248
R/boxplot.R             165     137  16.97%   41-65, 119-251
R/checkmate.R            18       9  50.00%   110-118
R/experimentSpec.R       92      58  36.96%   99, 214-285
R/forestplot.R          193     168  12.95%   60-91, 146-306
R/geneSpec.R            255     136  46.67%   153-168, 297-448
R/km.R                  182     151  17.03%   63-92, 152-297
R/pca.R                 344     261  24.13%   35-55, 162-449
R/quality.R             291     221  24.05%   18-109, 202-421
R/sampleVarSpec.R       221     100  54.75%   296-302, 314-321, 323, 331-342, 344-345, 347, 350, 358-362, 364-379, 384-407, 410-414, 416, 423-433, 435-436, 444, 489-506
R/scatterplot.R         160     130  18.75%   40-64, 121-248
R/utils.R                16       5  68.75%   74-78
R/volcanoplot.R         187     157  16.04%   35-56, 110-271
R/zzz.R                   1       1  0.00%    2
TOTAL                  2488    1710  31.27%

Results for commit: 63c2ac3

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Collaborator

@danielinteractive danielinteractive left a comment

Choose a reason for hiding this comment

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

thanks @nikolas-burkoff , sorry I was not clear I guess about it, can you do the gene names fix also for other gene selection in the other modules?

@nikolas-burkoff
Copy link
Contributor Author

thanks @nikolas-burkoff , sorry I was not clear I guess about it, can you do the gene names fix also for other gene selection in the other modules?

Yup

@nikolas-burkoff nikolas-burkoff merged commit 4ad782c into main Oct 7, 2022
@nikolas-burkoff nikolas-burkoff deleted the 284_reporter_fixes@main branch October 7, 2022 13:21
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.

[UAT] Test teal.reporter in modules
2 participants