JuliaComputing / xtrx_julia

XTRX LiteX/LitePCIe based design for Julia Computing
BSD 2-Clause "Simplified" License
1 stars 0 forks source link

Build out `XTRX.jl` #130

Closed staticfloat closed 2 years ago

staticfloat commented 2 years ago

The plan is to use XTRX.jl as a generic test harness for the XTRX, and we will run a set of regression tests (currently being built out in XTRX.jl/test/loopback_tests) on every commit. They will slowly become more and more exhaustive, until we can assert complete mastery over the device.

Currently, I have written up a good chunk of code to enable us to setup the device in a precise way for each test, but there will be more things we must assert. I imagine us writing test cases for getting/setting various values, analyzing the received signals, etc... Since these will be run for every PR, we shouldn't do hours-long tests here, only quick-to-execute tests, but we should still strive to be exhaustive.

staticfloat commented 2 years ago

@sjkelly Here's the PR I've been talking about for a while; I will probably not merge this until I actually have the CI bot working, so that I can run these tests (even if they fail) on CI.

There's still some work to be done to flesh out more of the ability to set up the XTRX and whatnot, but I hope you can start to see the gist of what I'm going for here.

sjkelly commented 2 years ago

I think we should merge the wrapper for LMS7002M-driver since @zsoerenm has been asking me about direct calls.