-
Notifications
You must be signed in to change notification settings - Fork 8
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
Feat/added the possibility to show the parameters within the table #244
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use in future branches the branch naming convention, which is lower-case snake_case
I was wondering why this has to be implemented for every estimator separately. I see it is because the table function is overridden in every estimator to enable doctests... maybe for now its ok, but generally we might want to think of something that is easier to maintain, if anyone has an idea.
I was hoping not. Maybe we could use the |
addressed issues by andre. ready for review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks nice. Should we add one or two doctests for this feature?
8888045
to
f32361d
Compare
f32361d
to
99db65d
Compare
|
Description
Test script: