-
Notifications
You must be signed in to change notification settings - Fork 22
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
Fully implement the print functions. #16
Comments
@codetalker7, any update on this issue? |
@sourish-cmi no update as such. I had just copied the old print messages (i.e from the old code) and put them in this file. For Bayesian models, we just use the For the frequentist models, the old code contained a few print statements; it'll be great if we can discuss proper printing messages for all these. |
@codetalker7, In the frequentist method, I think the output would be
I would like to see a similar table for Bayesian regression - something like this
|
I see. @sourish-cmi if possible, can you provide the Or rather, are the current ones in For the Bayesian models, we already have two tables printed out: |
@codetalker7 currently, the frequentist method prints You are correct. The Bayesian models have printed out all the necessary information. But if the |
The file for this issue is src/print.jl. We have extended the
Base.show
method to print details about models when they are trained; currently these details are not filled in appropriately, and might need some improvements.The text was updated successfully, but these errors were encountered: