[Prototype] Add mini.qubit for testing and reference
#5445
Conversation
|
Hello. You may have forgotten to update the changelog!
|
|
|
||
| """ | ||
|
|
||
| name = "mini.qubit" |
There was a problem hiding this comment.
Minor comment, but if you want to name it reference.qubit, it will help enforce that it is a reference device for future developers, and minimize users finding/using it :)
There was a problem hiding this comment.
@albi3ro any concerns with calling it reference.qubit? I also like reference more than mini - which could sound like it's for small scale simulations.
| Args: | ||
| wires (int, Iterable[Number, str]): Number of wires present on the device, or iterable that | ||
| contains unique labels for the wires as numbers (i.e., ``[-1, 0, 2]``) or strings | ||
| (``['ancilla', 'q1', 'q2']``). Default ``None`` if not specified. While this device allows |
There was a problem hiding this comment.
| (``['ancilla', 'q1', 'q2']``). Default ``None`` if not specified. While this device allows | |
| (``['auxiliary', 'q1', 'q2']``). Default ``None`` if not specified. While this device allows |
|
How shall we proceed with this PR? Is this something that could be put in the Q3 technical roadmap? |
I would 100% say this is something we should add at some point in the future, when we have time. Both because it allows us to have much more robust testing, and because it forms a useful jumping off spot for device developers. Q3 technical roadmap might be a good spot. |
Nice! Looks like [SC-65558] is the story. |
| name="mini.qubit", | ||
| ) | ||
| program.add_transform( | ||
| diagonalize_measurements, stopping_condition=supports_operation, name="mini.qubit" |
There was a problem hiding this comment.
| diagonalize_measurements, stopping_condition=supports_operation, name="mini.qubit" | |
| diagonalize_measurements, supported_base_obs=(qml.Z,), name="mini.qubit" |
| if tape.shots: | ||
| samples = sample_state(state, shots=tape.shots.total_shots, seed=seed) | ||
| results = tuple(mp.process_samples(samples, tape.wires) for mp in tape.measurements) |
There was a problem hiding this comment.
Do we need to add shot vector support here?
|
|
||
|
|
||
| @transform | ||
| def split_up_non_commuting(tape): |
There was a problem hiding this comment.
We can probably remove this now.
|
Completed by #6181 |
Context:
This exploration can potentially solve two issues:
Providing a stripped-down reference for plugin developers, helping distinguish between what is required for a device, and the extra features that
default.qubitprovides. Many of the performance optimizations and bonus features available indefault.qubitare not necessary for plugins, and make the source code more difficult to get through.Allowing testing on a feature-less device. Sometimes code will work on
default.qubit, but not any other device becausedefault.qubithas features that normal plugins do not. By testing withmini.qubit, we haveDescription of the Change:
There are two main parts to this draft:
mini.qubitdevicemini.qubitWork from two other branches are included:
fix_vmapandnon-commuting-expanding-fixes.Benefits:
Possible Drawbacks:
Related GitHub Issues: