Closed ezio-melotti closed 2 years ago
Very good.
On 5/29/22 03:12, Ezio Melotti wrote:
This PR adds tests for the matching kernel.
With grow and full trees and the default population of 10 the species went extinct, so I had to special case these two to use a higher population. I tried looking into conditionally parametrize the tests but it seems the only way to do it would be to manually unroll the parameters and have 9 different entries instead of 2 entries with 3 parameters each.
With these values all tests still pass in ~100s on my machine. You can view, comment on, or merge this pull request online at:
https://github.com/kstaats/karoo_gp/pull/63
-- Commit Summary --
- Add tests for the matching kernel.
-- File Changes --
A karoo_gp/test/data/log_test[m-f].txt (11) A karoo_gp/test/data/log_test[m-g].txt (11) A karoo_gp/test/data/log_test[m-r].txt (11) M karoo_gp/test/test_cli.py (7)
-- Patch Links --
https://github.com/kstaats/karoo_gp/pull/63.patch https://github.com/kstaats/karoo_gp/pull/63.diff
This PR adds tests for the matching kernel.
With grow and full trees and the default population of 10 the species went extinct, so I had to special case these two to use a higher population. I tried looking into conditionally parametrize the tests but it seems the only way to do it would be to manually unroll the parameters and have 9 different entries instead of 2 entries with 3 parameters each.
With these values all tests still pass in ~100s on my machine.