Add benchmark script for mlpack_decision_tree#30
Add benchmark script for mlpack_decision_tree#30ThyrixYang wants to merge 3 commits intomlpack:masterfrom ThyrixYang:master
Conversation
|
Can one of the admins verify this patch? |
|
Can you add a simple test for the mlpack dtc benchmark script? Take a look at: https://github.com/zoq/benchmarks/blob/master/tests/benchmark_kernel_pca.py for how the test could look like. |
|
Can one of the admins verify this patch? |
|
Hi @zoq , I'm confused that I didn't find how these test script is used? python3 -m unittest benchmark_xxx.pybut not worked. |
|
You can either run |
|
@mlpack-jenkins test this |
|
@rcurtin I can run the benchmark by "make run", and when I run the benchmark test locally: So I think mlpack_decision_tree benchmark passed the test. |
|
The current jenkins setup uses mlpack 2.2.2 where you have to pass a seperate labels file: So I think, what we can do here is to pass the labels file, and we should also update mlpack as soon as possible. |
|
@zoq I see. I added that feature when I was writing this benchmark script. Because all datasets(train file) in benchmark system use the last column as labels. And all other methods go with it well. So I think it's better to merge this pr after mlpack is updated and it passes the tests. |
|
Sounds reasonable, I'll update the mlpack package later today and rerun the test. |
|
Hmm, I think we can't update the mlpack package until the currently running mlpack benchmarks finish, unfortunately. I suppose it could be a few days then. |
|
If we update the package right now and run the |
|
@mlpack-jenkins test this |
related to #886 in mlpack