Skip to content

Fix bugs with use_*_template(), doc_show, and tidyselect::all_of()#134

Merged
jpdunc23 merged 17 commits intomainfrom
fix-template-bugs
Jan 9, 2023
Merged

Fix bugs with use_*_template(), doc_show, and tidyselect::all_of()#134
jpdunc23 merged 17 commits intomainfrom
fix-template-bugs

Conversation

@tiffanymtang
Copy link
Copy Markdown
Collaborator

@tiffanymtang tiffanymtang commented Dec 21, 2022

Closes #128
Closes #130
Closes #135

Also makes modifications to list_to_tibble_row() and list_to_tibble() helper functions when handling data frames so that data frames (and hence tibbles) are not unlisted.

* change functionality of save_experiment to save the active varying across experiment, not the base experiment
@tiffanymtang tiffanymtang changed the title Fix bugs in use_*_template() and doc_show Fix bugs with use_*_template(), doc_show, and tidyselect::all_of() Dec 22, 2022
@jpdunc23 jpdunc23 self-assigned this Dec 28, 2022
Copy link
Copy Markdown
Contributor

@jpdunc23 jpdunc23 left a comment

Choose a reason for hiding this comment

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

@tiffanymtang I'll work on the various TODOs I created during the review.

Comment thread R/utils.R Outdated
Comment thread R/experiment.R Outdated
Comment thread tests/testthat/test-checkpointing.R Outdated
Comment thread R/utils.R
@jpdunc23 jpdunc23 marked this pull request as draft January 5, 2023 23:19
@jpdunc23 jpdunc23 requested review from jpdunc23 and removed request for jpdunc23 January 9, 2023 21:57
@jpdunc23 jpdunc23 marked this pull request as ready for review January 9, 2023 21:57
@jpdunc23 jpdunc23 added this to the v0.0.3 milestone Jan 9, 2023
@jpdunc23
Copy link
Copy Markdown
Contributor

jpdunc23 commented Jan 9, 2023

There are inconsistent failures in test-distributed.R, appearing either on ubuntu-latest (release) (here) or ubuntu-latest (devel) (here), but not both at the same time. Maybe related to #144? Except that the permission denied errors are gone in the latest run.

@jpdunc23 jpdunc23 linked an issue Jan 9, 2023 that may be closed by this pull request
4 tasks
@jpdunc23 jpdunc23 marked this pull request as draft January 9, 2023 22:21
@jpdunc23 jpdunc23 marked this pull request as ready for review January 9, 2023 23:18
@jpdunc23 jpdunc23 merged commit 979dfee into main Jan 9, 2023
@jpdunc23 jpdunc23 deleted the fix-template-bugs branch January 9, 2023 23:20
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.

Bug in set_doc_options() for vary across experiment use_*_template() bug: xy_dgp_constructor() missing tidyselect::all_of() deprecation warning

2 participants