Skip to content

add restrict_facets and export make_subset_expr#1081

Merged
gmbecker merged 9 commits intomainfrom
1080_restrict_facets
Apr 20, 2026
Merged

add restrict_facets and export make_subset_expr#1081
gmbecker merged 9 commits intomainfrom
1080_restrict_facets

Conversation

@gmbecker
Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions
Copy link
Copy Markdown
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: c13b18d

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 23, 2026

Unit Tests Summary

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

Results for commit c13b18d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 23, 2026

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
Pagination 💔 $16.31$ $+1.77$ $0$ $0$ $0$ $0$
Tabulation framework 💔 $23.91$ $+1.62$ $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.21$ restrict_facets_works
make-subset_expr 👶 $+0.02$ make_subset_expr_works

Results for commit 5100850

♻️ This comment has been updated with latest results.

@gmbecker
Copy link
Copy Markdown
Collaborator Author

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

Comment thread R/make_split_fun.R
Comment thread R/make_subset_expr.R
Comment thread R/make_subset_expr.R
Copy link
Copy Markdown
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
Copy Markdown
Collaborator Author

added some basic examples as requested

Copy link
Copy Markdown
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.

For me is good to go! Thanks Gabe :)

@gmbecker gmbecker merged commit df07078 into main Apr 20, 2026
29 of 30 checks passed
@gmbecker gmbecker deleted the 1080_restrict_facets branch April 20, 2026 15:05
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants