update tests for basic gates #300
Conversation
* Unify various DumpDiffs into one, more parametrized method * Clean up presentation of several tasks, add (hopefully) helpful messages
There was a problem hiding this comment.
This change should improve the user's debugging experience a lot, thank you for working on it!
I looked into how to unify the multiple versions of DumpDiff that use more-or-less the same logic, and by the time I convinced myself that my idea works it was pretty much implemented, so I went ahead and finished it. Passing the number of qubits and the state prep operation as parameters to the DumpDiff, combined with some wrappers for state prep gates, did the trick!
I also did a bit of cleanup on the tests we're running - for example, for task 2.1 we need to show the effect of the solutions the state we promised, not a general 2-qubit state, and the task 1.10 looked too heavy with controlled version of 2-qubit gate, after all. It looks like I did a pretty big revamp of your change, but it is mostly due to the unification of DumpDiff.
Thank you again for working on this!
#176