Closed drasmuss closed 5 years ago
Sweet! I just glanced at it, so this isn't an official review, but it looks good. Also seems fairly straightforward compared to what I feared, particularly when you started talking about having to patch nxsdk.
The now-merged #204 parallelized the hardware tests, but I don't believe the caching was included in that PR, so I'm leaving this open for now.
https://github.com/nengo/nengo-loihi/pull/219 added the caching, so this has all been done now
Two improvements to test time. First, use pytest-xdist to run tests in parallel. This has the most significant impact on the hardware tests, as we can run jobs on different boards simultaneously. Second, uses TravisCI's caching functionality to reduce the amount of time we spend setting up the environment for each job. This will be slower for a cache miss, but should hopefully improve things overall (depends on how often the environment changes); we should keep an eye on it to make sure that we aren't getting too many cache misses.