-
Notifications
You must be signed in to change notification settings - Fork 4
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.
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
Implement IO-Test for the MOM Operation #9
Comments
We should be able to write "offline" tests for this, testing only the moment calculation, provided raw io works. The reason we don't have that in down sampling and repetitive signal averaging is to test the measurement windows (masks) which raw io doesn't have |
As far as I know, the DAC-C-library performs this operations directly on the data obtained from the acquisition card. An offline test would require the library to perform the operations on data input via software. Or am I misunderstanding you here? |
No, you are right. I'll have a look at the C source tomorrow and check wether it is isolated enough to test on its own |
Maybe this could be realized as a unit test in the C(++) project. Then the library needs not to expose methods to perform operations on software input and the test would not have to bother with setting up the hardware... |
That is exactly what i was thinking about. as long as downsampling and the different mask types work (with the hardware), the processing could be tested individually in C++. summoning @terrorfisch: What do you think |
Hmm, if we take that thought one step further, downsampling and repetitive signal averaging should also be testable (?) in the C++ project, if they are not too tightly coupled to the hardware input.. |
@terrorfisch has started writing tests for the data acquisition driver. |
The DAC driver offers an operation to obtain the raw moment of the downsampled data.
Tests for this operation for different orders of this moment should be implemented.
The text was updated successfully, but these errors were encountered: