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

Unit test: we need a print for each ParamSet for a learner #1352

Closed
berndbischl opened this issue Nov 21, 2016 · 0 comments
Closed

Unit test: we need a print for each ParamSet for a learner #1352

berndbischl opened this issue Nov 21, 2016 · 0 comments
Assignees
@florianfendt
Labels
prio-medium

Comments

@berndbischl
Copy link
Sponsor Member

We had a bug here #1292

Such a test will catch this.

we can do this in testThatLearnerCanTrainPredict, maybe the name of the function could be generalized then.
@berndbischl berndbischl mentioned this issue Dec 16, 2016
Closed
florianfendt added a commit that referenced this issue Jan 9, 2017
@florianfendt
test for print.ParamSet closes #1352
71a55cf
smilesun pushed a commit that referenced this issue Feb 9, 2017
@florianfendt @smilesun
adds test for print.ParamSet. Closes #1352 (#1419)
923b998 
* test for print.ParamSet closes #1352

* rm unwanted bin file

* adjust other tests in learners_all

* update function docu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@florianfendt florianfendt

Labels
prio-medium
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@larskotthoff @berndbischl @florianfendt