Conversation
…junco`\] package documentation.
* Fix bug in s_summarize_desc_j function to handle sparse data (zero variance) cases (#78) (#79) * hotfix + test * news and upgrade to 0.1.1.1 for consistency sake * refactor name * create new snapshot test and S4 expect class * added CHANGELOG.md * [skip style] [skip vbump] Restyle files * fix: rcmdcheck notes and spellcheck * [skip style] [skip vbump] Restyle files * fix: rbuildignore * fix: linting issues --------- Co-authored-by: David Muñoz Tord <david.munoz@mailbox.org> * Main hotfix nov 2025 (#82) * hotfix: fix #77 (hdrmpf typo) and fix too long (topleft bug fix) #80 * hotfix: fix #31 (h_subset_combo handles NA values in flag_var correctly) * minor update: changed pull request template -> added unit tests check * fix: test-tt_to_tblfile.R rollback to main version + new final regression test * Revert "Main hotfix nov 2025" (#83) * Revert "Main hotfix nov 2025 (#82)" This reverts commit baa4196. * [skip style] [skip vbump] Restyle files --------- Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> * Update pkgdown.yaml (#101) update to pkgdown 2.2.0 * fix: lint * add: snaps * remove: old pipeline * fix: lintr * update: README * bump: verison 0.1.3 * update: CI/CD * update: ../reference/index.html * update: A complete list of analysis functions can be found in the \[`junco`\] package documentation. * update: rhub * update: rhub * make it shorter * 0.1.bump: 4 --------- Co-authored-by: Ilse <55379552+iaugusty@users.noreply.github.com> Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
# Conflicts: # .github/workflows/check_CRAN.yml # CHANGELOG.md # NEWS.md # README.md # cran-comments.md # inst/WORDLIST
* #' @importFrom tern get_covariates * leftside export * postfun_eq5d export * ac_blank_line export * update CHANGELOG.md * update pkgdown.yml
* started code refactoring * second iteration of code refactoring * replaced the snapshot tests of docx files with the docx_summary() output dataframes * linted file * Update docx_exporter_functions.R * updated documentation, removed unused arguments * Update CHANGELOG.md * Update docx_exporter_functions.R * Update docx_exporter_functions.R * Update CHANGELOG.md * minor bugfixes * docs: removing the junco:: prefixes --------- Co-authored-by: munoztd0 <david.munoztord@mailbox.org>
* addresses #138 , hanging indentation in first column is now accomplished by manipulating the XML, removed dependency of 'systemfonts' package, removed functions wrap_string_with_indent() and add_hanging_indent_first_column() * linted, updated snapshots * minor bugfix in the calculation of the left indentation, updated snapshots --------- Co-authored-by: David Muñoz Tord <david.munoz@mailbox.org>
# Conflicts: # CHANGELOG.md # DESCRIPTION # NEWS.md
# Conflicts: # CHANGELOG.md # DESCRIPTION # NEWS.md
* debug ci/cd * debug ci/cd * Refactor pkgdown workflow configuration * Update GitHub Actions workflow for R-CMD-check * Update check-CRAN workflow configuration * Add name and permissions to pkgdown workflow
* Update table_and_listing_customizations.rmd * Update CHANGELOG.md
#172) * fixed tt_to_flextable_j() to have correct left-indentation in Header col 1, and fixed Footers to have first line with linebreak * Update CHANGELOG.md
* first iteration of merging functions + added checkmate checks + updated documentation * merged export_as_docx_j() and export_graph_as_docx() into export_TLG_as_docx() * Update CHANGELOG.md * added export_TLG_as_docx() in _pkgdown.yml, updated vignette, minor bugfixes * minor bugfix * Update docx_exporter_functions.R --------- Co-authored-by: David Muñoz Tord <david.munoz@mailbox.org>
* addresses #142 * Update tt_to_tblfile.R * added option to export csv in export_as_docx_j() * added option to export csv to export_as_docx_j() function, added unit tests * Update CHANGELOG.md * linted test-docx_exporter_functions.R * Update tt_to_tblfile.R * minor changes in typos * tests: add tests for for tt_to_tlgrtf() export behavior * fix: lintr * in export_as_docx_j() set the default value for argument export_csv = FALSE * argument export_csv set default to FALSE in tt_to_tlgrtf() * Update test-tt_to_tlgrtf_output.R * Update tt_to_tblfile.R --------- Co-authored-by: munoztd0 <david.munoztord@mailbox.org> Co-authored-by: David Muñoz Tord <david.munoz@mailbox.org>
* Update docx_exporter_functions.R * Update docx_exporter_functions.R * Update CHANGELOG.md --------- Co-authored-by: David Muñoz Tord <david.munoz@mailbox.org>
# Conflicts: # CHANGELOG.md # DESCRIPTION # NEWS.md # R/docx_exporter_functions.R # tests/testthat/_snaps/docx_exporter_functions.md # tests/testthat/test-docx_exporter_functions.R
* start with what needs to be exported * refresh docs * Update CHANGELOG.md for Export helpers for LS means tabulation --------- Co-authored-by: Daniel Sabanes Bove <DBove@ITS.JNJ.com> Co-authored-by: David Muñoz Tord <david.munoz@mailbox.org>
* prepare for cran release * don't export files when on cran in vignette * 196 feature request make overall tests shorter for cran (#198) * skip flextable * passed the uneccessary sanity check to on_cran = FALSE * the actual rtf and docx output generation is the bottleneck, each function call takes about 5 secs, and we have 26 of them .. (as per @jkovach2 request) so I suggest to remove from .Rbuildignore because those feature are already checked in test-tt_to_tblfile.R * test-tt_to_tblfile.R: made wide Tables less wide to generate fewer RTFs * removed annoying messages in 'test-a_summarize_aval_chg_diff.R' and 'test-summarize_mmrm.R' * make export not run on CRAN --------- Co-authored-by: Ezequiel Anokian <lainjoso@hotmail.com> --------- Co-authored-by: Gabe Becker <gabembecker@gmail.com> Co-authored-by: Ezequiel Anokian <lainjoso@hotmail.com>
* update processing denominator dataset for combined facet in relative risk * add tests for relative risk difference column in a combined column * update CHANGELOG
#205) * decreased the unit tests to 40 secs, same as tt_to_tlgrtf() unit tests * Update CHANGELOG.md
* s_proportion_diff_j removed * replace tern function * replace old with new tern function * remove all tests * fixed documentation * update news * add a test for label * Apply suggestion from @munoztd0 * remove old param --------- Co-authored-by: David Muñoz Tord <david.munoz@mailbox.org>
* Remove explicit .alt_df_full from extra_args * clean up * update def prog * fix
…e prop diff s/a functions based on the MF criterion.
Solves 498: Set default `RefFootnote()` and add `mf_threshold` argument for conditional prop diff a/s funs.
* initial commit * updates to default formats * roxygen + tests * review file for default formats * default format count_fraction to jjcsformat_count_fraction * - update stat mean_diffci into mean_diff_with_ci - a_summary_subset afun to deal with fmt_d as format input * utilize formats_var in a_summary_subset * refactor column_stats and vars_in_cols_setup (former postfun_eq5d) * renamed vars_in_cols_setup into stats_in_cols_setup * spelling and styler * add fmt_spec_d and fmt_utils pages to pkgdown.yml * resolve check-r-package issues * lintr + description import * resolve check problems * add dplyr call in examples * resolve tribble issues in roxygen examples * resolve example problems * resolve check() issues * allow format setup for format_sigfig_j with varying precision * to prevent dependency problems * add extra tests * add vignette + some smaller updates * remove junco::: as fmt_spec_single_d is now exported * resolve merge conflict * resolve merge conflict * resolve merge conflict * spelling + update links * update link * small typos * diff_ stats are from proportions estimation, default to report to 1 decimal place, not 2 * draft vignette for PK formatting function * this will not EVAL in cran and allow random.cdisc.dat to be in suggest * fix cross linking * fix note * address review feedback Joe * to avoid hand calculation of number of d-style stats in junco_def_d_all * spelling correction * add 4th example for stats in columns --------- Co-authored-by: David Muñoz Tord <david.munoz@mailbox.org> Co-authored-by: munoztd0 <dtord@its.jnj.com>
…rowwise 500: Add `method_scope` argument to `cond_proportion_j` functions
* add test for a_freq_j using cmh sato where tern bug occured * Trigger CI * per feedback limit test for bug to a_freq_j only
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
Test for release 0.1.7
Fixes #496