Skip to content

add restrict_facets and export make_subset_expr#1081

Open
gmbecker wants to merge 4 commits intomainfrom
1080_restrict_facets
Open

add restrict_facets and export make_subset_expr#1081
gmbecker wants to merge 4 commits intomainfrom
1080_restrict_facets

Conversation

@gmbecker
Copy link
Collaborator

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 23, 2026

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
R/00tabletrees.R               873      68  92.21%   20, 72-76, 114, 117, 448, 539-540, 543, 705, 811, 938-939, 1041, 1044, 1046-1047, 1065-1068, 1088, 1203-1206, 1304-1309, 1472, 1573-1576, 1700-1703, 1740-1743, 1749-1754, 1814, 1821, 1917, 2029, 2042, 2045-2048, 2051-2054, 2084, 2117-2118
R/as_html.R                    172      25  85.47%   5-10, 80, 152-157, 162-167, 182-186, 273
R/colby_constructors.R         626      36  94.25%   81, 134, 197-200, 267-270, 411, 427, 1211-1215, 1217-1221, 1302, 1391, 1552, 1591, 1602, 1610, 1613, 1638, 1659, 1805, 2028-2031
R/compare_rtables.R             83      17  79.52%   93-96, 99-102, 115-118, 137, 156-157, 188, 193
R/custom_split_funs.R          265      40  84.91%   127, 132, 138-143, 156, 173-177, 353-358, 375-380, 456, 502, 518-521, 537, 599, 609-610, 612, 624, 668, 693
R/default_split_funs.R         287      22  92.33%   272, 335-338, 349-350, 352, 354, 551-555, 619-622, 685-688
R/format_rcell.R                17       1  94.12%   47
R/indent.R                      13       2  84.62%   40-41
R/index_footnotes.R             66       0  100.00%
R/make_split_fun.R             166      30  81.93%   22-26, 36-39, 52-55, 58-61, 115, 119, 267, 270-273, 278-281, 366, 375, 377, 379, 430
R/make_subset_expr.R           137      14  89.78%   77-92, 169-177, 213, 302, 306, 315
R/summary.R                    144      38  73.61%   35, 80, 178-220, 269, 315-331, 366, 397
R/tree_accessors.R            1280     147  88.52%   110, 139-140, 264, 284, 310, 333, 363, 381, 400-404, 424, 446-449, 576, 603-604, 890-896, 1040, 1059, 1085, 1137, 1194-1195, 1236, 1271, 1309-1314, 1373, 1447-1451, 1469-1478, 1556, 1676-1679, 1704, 1726-1727, 1737, 1788, 1809-1814, 1835-1840, 1925, 1966, 2065, 2172, 2185, 2199, 2215, 2224, 2234-2238, 2288-2293, 2496, 2506-2509, 2519, 2544-2547, 2554, 2556-2559, 2681, 2715-2716, 2773, 3077, 3438, 3554, 3588-3613, 3704-3712, 3873, 3947-3953, 4258, 4382, 4467-4472, 4478, 4502-4507, 4555, 4580-4604, 4633-4639
R/tt_afun_utils.R              419      33  92.12%   60, 182, 189, 198-212, 280, 288-289, 507, 515-518, 600-604, 624, 638-640
R/tt_as_df.R                   400      23  94.25%   101-104, 112, 150, 224-227, 369, 388, 458, 477-480, 489, 599, 605, 637, 655, 707
R/tt_compare_tables.R           72       4  94.44%   51, 174, 249, 253
R/tt_compatibility.R           574      70  87.80%   22, 149-150, 193, 198, 329-330, 334-337, 343, 347, 531, 585-588, 625-627, 665, 698, 718, 738-741, 751-754, 799, 816-820, 826-829, 903, 930-933, 942, 1004, 1012, 1023-1026, 1137, 1144, 1172-1186, 1217-1218
R/tt_dotabulation.R           1266      97  92.34%   60, 255, 260, 262, 311, 336, 340-343, 376-379, 402, 435-438, 466-469, 596-597, 665, 852-856, 906, 910, 938-941, 951, 971-975, 982-985, 1249, 1253, 1284, 1388-1391, 1605-1613, 1877-1886, 1968-1971, 1982, 1987, 1992-1993, 1995, 2006, 2011, 2034, 2120-2139
R/tt_export.R                   13       1  92.31%   45
R/tt_from_df.R                  15       0  100.00%
R/tt_paginate.R                535      40  92.52%   74, 122-131, 242, 341-342, 494, 629-632, 653-657, 802-805, 856-863, 940, 943, 961, 968, 971
R/tt_pos_and_access.R          656      33  94.97%   76, 78-80, 105, 166, 262, 329, 438, 512, 516, 724, 726, 734, 740, 754, 764-767, 990, 1007-1010, 1037, 1096-1097, 1110, 1346-1347, 1373-1376, 1650, 1725
R/tt_showmethods.R             162      21  87.04%   56, 91-113, 223, 249, 258, 263, 266-270, 359-360
R/tt_sort.R                    115       6  94.78%   50, 289-292, 300
R/tt_toString.R                439      24  94.53%   125, 355, 377, 390, 400, 406, 409, 415-425, 518, 619, 826-851
R/utils.R                       34       7  79.41%   56, 169-174
R/validate_table_struct.R       84      10  88.10%   80-84, 93-94, 140, 149-150
R/Viewer.R                      61       9  85.25%   46, 50, 60-64, 84, 118
TOTAL                         8974     818  90.88%

Diff against main

Filename                Stmts    Miss  Cover
--------------------  -------  ------  -------
R/make_split_fun.R        +28      -1  +4.39%
R/make_subset_expr.R        0      -1  +0.73%
R/tree_accessors.R          0      -1  +0.08%
TOTAL                     +28      -3  +0.06%

Results for commit: 1b14bff

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions
Copy link
Contributor

github-actions bot commented Mar 23, 2026

Unit Tests Summary

    1 files     30 suites   1m 58s ⏱️
  249 tests   249 ✅ 0 💤 0 ❌
1 856 runs  1 856 ✅ 0 💤 0 ❌

Results for commit 1b14bff.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 23, 2026

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
Result Data Frames 💚 $12.55$ $-1.08$ $0$ $0$ $0$ $0$
make-subset_expr 👶 $+0.02$ $+7$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
Custom Split Functions 👶 $+0.24$ restrict_facets_works
make-subset_expr 👶 $+0.02$ make_subset_expr_works

Results for commit 03a8f92

♻️ This comment has been updated with latest results.

@gmbecker
Copy link
Collaborator Author

ah my bad, I styled R/ but forgot to do the files in tests.

Comment on lines +468 to +470
#'
#' @seealso [make_split_fun()]
#'
Copy link
Contributor

Choose a reason for hiding this comment

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

I would add an example

#' @return A subseting expression to be used to restrict data to a
#' particular column during tabulation.
#' @export

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

Comment on lines +23 to +24
#' @return A subseting expression to be used to restrict data to a
#' particular column during tabulation.
Copy link
Contributor

Choose a reason for hiding this comment

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

example also here

Copy link
Contributor

@Melkiades Melkiades left a comment

Choose a reason for hiding this comment

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

I think it is good to go! Thanks for the additions. I would only add a couple of examples to the documentation

@gmbecker
Copy link
Collaborator Author

added some basic examples as requested

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