RECETOX / recetox-aplcms

This is a custom fork of apLCMS containing adaptations for large scale studies.
GNU General Public License v2.0
4 stars 8 forks source link

Small refactorings on combine.seq.3 and general adaptive.bin #196

Closed hechth closed 1 year ago

hechth commented 1 year ago

Refactored the combine.seq.3 function to only use dplyr to summarize the data (as this is what it actually does) and generally renamed some variables in adaptive.bin and proc.cdf

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.53 :warning:

Comparison is base (463e1cd) 66.65% compared to head (4911868) 66.13%.

:exclamation: Current head 4911868 differs from pull request most recent head a555966. Consider uploading reports for the commit a555966 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #196 +/- ## ========================================== - Coverage 66.65% 66.13% -0.53% ========================================== Files 23 23 Lines 2624 2616 -8 ========================================== - Hits 1749 1730 -19 - Misses 875 886 +11 ``` | [Impacted Files](https://codecov.io/gh/RECETOX/recetox-aplcms/pull/196?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RECETOX) | Coverage Δ | | |---|---|---| | [R/compute\_clusters.R](https://codecov.io/gh/RECETOX/recetox-aplcms/pull/196?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RECETOX#diff-Ui9jb21wdXRlX2NsdXN0ZXJzLlI=) | `81.63% <ø> (ø)` | | | [R/adaptive.bin.R](https://codecov.io/gh/RECETOX/recetox-aplcms/pull/196?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RECETOX#diff-Ui9hZGFwdGl2ZS5iaW4uUg==) | `100.00% <100.00%> (ø)` | | | [R/aggregate\_by\_rt.R](https://codecov.io/gh/RECETOX/recetox-aplcms/pull/196?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RECETOX#diff-Ui9hZ2dyZWdhdGVfYnlfcnQuUg==) | `100.00% <100.00%> (ø)` | | | [R/recover.weaker.R](https://codecov.io/gh/RECETOX/recetox-aplcms/pull/196?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RECETOX#diff-Ui9yZWNvdmVyLndlYWtlci5S) | `98.32% <100.00%> (ø)` | | | [R/remove\_noise.R](https://codecov.io/gh/RECETOX/recetox-aplcms/pull/196?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RECETOX#diff-Ui9yZW1vdmVfbm9pc2UuUg==) | `58.33% <100.00%> (-22.92%)` | :arrow_down: | | [R/run\_filter.R](https://codecov.io/gh/RECETOX/recetox-aplcms/pull/196?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RECETOX#diff-Ui9ydW5fZmlsdGVyLlI=) | `100.00% <100.00%> (ø)` | | | [R/utils.R](https://codecov.io/gh/RECETOX/recetox-aplcms/pull/196?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RECETOX#diff-Ui91dGlscy5S) | `90.12% <100.00%> (ø)` | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RECETOX). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RECETOX)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.