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.

Sign up for GitHub

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

Jump to bottom

Replace print-results with verbose #129

Open
pzivich opened this issue Nov 5, 2019 · 1 comment
Open

Replace print-results with verbose #129

pzivich opened this issue Nov 5, 2019 · 1 comment
Assignees
@pzivich
Labels
Causal inference Updates for the causal inference branch Docs Updates to the docs / print functions Intermediate Issues/additions that will be completed relatively soon
Projects
For 1.0.0 release

Comments

@pzivich
Copy link
Owner

pzivich commented Nov 5, 2019

Currently, many arguments have a print_results optional argument which will provide statsmodels output for fit parametric models. Instead of this, I should replace with a new argument in _init_().

The new argument would be verbose, which when set to True would print all available parametric model fitting info (maybe some additional information as well depending on the estimator). Argument would default to False.

This would make it easier than controlling each print_results flag and would allow for the printing of additional information to the console.
@pzivich pzivich added Intermediate Issues/additions that will be completed relatively soon Docs Updates to the docs / print functions Causal inference Updates for the causal inference branch labels Nov 5, 2019
@pzivich pzivich self-assigned this Nov 5, 2019
@pzivich pzivich added this to Future-Changes in For 1.0.0 release Jan 9, 2020
@pzivich
Copy link
Owner Author

pzivich commented Jan 9, 2020

StochasticTMLE which will be released in v0.8.3 follows the new verbose setup. All newly implemented estimators will follow this structure. For the release of v1.0.0, all other estimators will make the switch to the new verbose argument

@pzivich pzivich mentioned this issue Jul 12, 2020
Merged
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pzivich pzivich

Labels
Causal inference Updates for the causal inference branch Docs Updates to the docs / print functions Intermediate Issues/additions that will be completed relatively soon
Projects
For 1.0.0 release
Future-Changes
Milestone
No milestone
Development

No branches or pull requests
1 participant
@pzivich