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

BART random variable args #43

Open
GStechschulte opened this issue Jan 8, 2025 · 0 comments
Open

BART random variable args #43

GStechschulte opened this issue Jan 8, 2025 · 0 comments
Labels
enhancement New feature or request question Further information is requested

Comments

@GStechschulte
Copy link
Owner

See below a conversation between Osvaldo and I:

I think people get confused by the need to pass Y to the BART RV. One alternative could be to change the name from Y to "init_something" so it becomes more clear that we are using that for initialization and that it just happens that Y usually is a good idea. Another one will be to not only rename "Y" argument but also split it. "init_mean" for the mean of the sum of trees and "init_scale" (if we learn the scale) or "prior_leaf_sd" (if we use a prior).

I think renaming and splitting it would be a good option

@GStechschulte GStechschulte added enhancement New feature or request question Further information is requested labels Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant