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

Increase character length in deparse calls #20

Open
orourabi opened this issue Oct 31, 2023 · 0 comments
Open

Increase character length in deparse calls #20

orourabi opened this issue Oct 31, 2023 · 0 comments

Comments

@orourabi
Copy link

The number of confounders / length of confounder names is currently being limited by the call to deparse here

This results many confounders being deparsed to "one + two + three + four + five + six + seven + eight + nine + " " ten" or silly long names to "really_long_variable_name + another_really_long_variable_name + " " a_third_long_variable_name" which causes an error of:

Error in if (grepl("(?<![\\w\\.])\\.(?![\\w\\.])", deparse(matchedCall$confounders), :
the condition has length > 1

I think if the width.cutoff parameter was increased in the deparse call, this would avoid this problem but I haven't checked if there would be more problems downstream.

This is probably an odd use case to have many confounders or confounders with long names, so maybe isn't a priority fix, or maybe you know a workaround (this is my first time trying to use this function!)

Thanks for making so many useful BART-related packages!

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

No branches or pull requests

1 participant