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

Remove references to is.bool and is.scalar #686

Conversation

Puzzled-Face
Copy link
Collaborator

Pull Request

Fixes #nnn

@github-actions
Copy link
Contributor

github-actions bot commented Oct 2, 2023

badge

Code Coverage Summary

Filename                    Stmts    Miss  Cover    Missing
------------------------  -------  ------  -------  ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
R/checkmate.R                  49       0  100.00%
R/crmPack-package.R             4       0  100.00%
R/Data-class.R                 80       4  95.00%   455-456, 460-465
R/Data-methods.R              202       0  100.00%
R/Data-validity.R             144       1  99.31%   21
R/Design-class.R               75       0  100.00%
R/Design-methods.R           2742    2317  15.50%   110, 135-138, 146-157, 161-180, 192-196, 202-215, 373-934, 999-1340, 1391, 1581, 1751-4544, 4663
R/Design-validity.R            20       0  100.00%
R/fromQuantiles.R             178      69  61.24%   237-378
R/helpers_covr.R               23       0  100.00%
R/helpers_data.R               84       1  98.81%   139
R/helpers_design.R             75      36  52.00%   22, 77-129
R/helpers_jags.R               77       0  100.00%
R/helpers_model.R              78       4  94.87%   38, 89-90, 139
R/helpers_rules.R             447       0  100.00%
R/helpers.R                   271     103  61.99%   103-118, 142-147, 162, 175-189, 225-422, 452-501
R/logger.R                     11       0  100.00%
R/mcmc.R                      287      18  93.73%   90-95, 374-375, 385, 387-388, 391-394, 577-578, 667, 673, 731
R/McmcOptions-class.R          18       0  100.00%
R/McmcOptions-methods.R         8       1  87.50%   43
R/McmcOptions-validity.R       42       0  100.00%
R/Model-class.R               960     155  83.85%   135-137, 205-207, 211-213, 272-274, 346-348, 352-354, 433-435, 502-504, 566-570, 573-576, 679-682, 686-687, 802-806, 926-928, 932-940, 1085-1087, 1092-1095, 1099-1102, 1218-1222, 1224-1227, 1231-1234, 1237, 1398-1408, 1413-1419, 1574-1577, 1583-1590, 1747, 1756, 1765, 1774, 1783-1788, 1924, 1933, 1942, 1950-1952, 2691-2720, 2724-2730, 2737-2741, 2746, 2851-2864, 2890, 2986-2988, 2992, 3085-3087, 3091, 3160-3172, 3190
R/Model-methods.R             376      34  90.96%   78, 155, 208, 675-720, 1039-1048
R/Model-validity.R            435       8  98.16%   430-433, 442-445
R/ModelParams-class.R          13       0  100.00%
R/ModelParams-validity.R       21       0  100.00%
R/Rules-class.R               392       2  99.49%   3068, 3130
R/Rules-methods.R            1429     164  88.52%   741, 744, 887, 890, 893, 1008, 1011, 1014, 1134-1137, 1171, 1274-1277, 1312, 2475-2483, 2507-2514, 3017-3260
R/Rules-validity.R            418       0  100.00%
R/Samples-class.R               1       0  100.00%
R/Samples-methods.R          1104      45  95.92%   406-416, 644, 1358-1393, 1616, 1629, 1794, 2124-2129
R/Samples-validity.R           10       0  100.00%
R/Simulations-class.R          49      32  34.69%   470-819
R/Simulations-methods.R      1597    1454  8.95%    65-350, 406, 416-435, 448-453, 500-509, 675-2935
R/utils.R                       6       0  100.00%
TOTAL                       11726    4448  62.07%

Diff against main

Filename              Stmts    Miss  Cover
------------------  -------  ------  -------
R/Design-methods.R      -20     -16  -0.03%
R/helpers.R              -6       0  -0.82%
R/Model-class.R          +6       0  +0.10%
TOTAL                   -20     -16  +0.07%

Results for commit: 00b561c

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 @Puzzled-Face , see suggestions inside which also apply to other changes

R/Design-methods.R Outdated Show resolved Hide resolved
R/Design-methods.R Outdated Show resolved Hide resolved
R/helpers_model.R Outdated Show resolved Hide resolved
…tions-from-checkmate' of github.com:Roche/crmPack into 685-replace-isscalar-and-isbool-with-corresponding-functions-from-checkmate
@Puzzled-Face Puzzled-Face mentioned this pull request Oct 2, 2023
16 tasks
@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Unit Tests Summary

       1 files       36 suites   3m 43s ⏱️
1 173 tests 1 045 ✔️ 128 💤 0
6 190 runs  6 027 ✔️ 163 💤 0

Results for commit 4c484ee.

♻️ This comment has been updated with latest results.

@Puzzled-Face Puzzled-Face merged commit d6d5434 into main Oct 4, 2023
20 checks passed
@Puzzled-Face Puzzled-Face deleted the 685-replace-isscalar-and-isbool-with-corresponding-functions-from-checkmate branch October 4, 2023 07:53
Puzzled-Face added a commit that referenced this pull request Oct 16, 2023
* Refactor setSeed and getResultList (#676)

Co-authored-by: Daniel Sabanes Bove <danielinteractive@users.noreply.github.com>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

* 675: Add `DesignGrouped` (#677)

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

* Fixes first cohort logic for grouped design simulation (#684)

* Remove references to is.bool and is.scalar (#686)

* Remove references to is.bool and is.scalar

* [skip actions] Restyle files

* Responding to review comments

* Removing lintr moan

* Updating documentation

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

* Remove plot.gtable and print.gtable (#697)

* Remove plot.gtable and print.gtable

* Fix _pkgdown.yaml

* 690 remove safeinteger and iswholenumber 1 (#693)

* Remove safeInteger and is.wholenumber

* Fixing false test failures

* [skip actions] Restyle files

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

* Remove noOverlap (#695)

* remove %~% (#689)

* remove %~%

* Fixing NAMESPACE

* 699 remove multiplot (#700)

* Remove multiplot

* Correct typo

* Correcting typo in argument to assert_integerish

* New commit for additional statistics (#663)

* New commit for additional statistics

* [skip actions] Restyle files

* Update: additional statstics added to other designs bug correction suggetions by Clara

* bug corrected

* Update: Additional_stats

* update: additional_stats

* updaze: error_corrections (additional_stats)

* update: additional_statistics

* Dynamic: Additional_statistical_summary

* Additional statistical summary function update

* additional statistical summary changes

* Update: Additional Statistical Summary

* small change: additional stat summary

* Suggested changes: additional_statistics and validation

* [skip actions] Restyle files

* Simulation refectoring

* Simulation small update

* [skip actions] Restyle files

* small update

* incorrectly named file:delete

* removing documentation errors

* update yaml file

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Clara <clara.beck@bayer.com>
Co-authored-by: Oliver Boix <95433070+0liver0815@users.noreply.github.com>
Co-authored-by: Daniel Sabanes Bove <danielinteractive@users.noreply.github.com>

* Refactor myBarplot (#703)

* Refactor myBarplot

* [skip actions] Restyle files

* Responding to lintr complaints

* Rsponding to review comments

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: Oliver Boix <95433070+0liver0815@users.noreply.github.com>
Co-authored-by: Daniel Sabanes Bove <danielinteractive@users.noreply.github.com>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Prerana Chandratre <82475780+Prerana17@users.noreply.github.com>
Co-authored-by: Clara <clara.beck@bayer.com>
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.

Replace is.scalar and is.bool with corresponding functions from checkmate
2 participants