-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Open
Labels
area/channelsgood for learningFor beginners in QC, this will help picking up some knowledge. Bit harder than "good first issues"For beginners in QC, this will help picking up some knowledge. Bit harder than "good first issues"good part-time projectA meaty non-urgent issue with a substantial amount of work to be done.A meaty non-urgent issue with a substantial amount of work to be done.kind/design-issueA conversation around designA conversation around designtriage/acceptedA consensus emerged that this bug report, feature request, or other action should be worked onA consensus emerged that this bug report, feature request, or other action should be worked on
Description
Most (all?) our channels are specified via _kraus_
method that returns a sequence of Kraus operators. This is inefficient for channels such as Reset whose Kraus operators are sparse and whose action is easy to describe in terms of changes they make to the elements of the input density matrix. IOW, such channels should be implemented using _apply_channel_
, not _kraus_
.
Filing for discussion, identification of the offending channels and subsequent implementation changes.
Context: Review of #5851 (which adds yet another offending channel).
Metadata
Metadata
Assignees
Labels
area/channelsgood for learningFor beginners in QC, this will help picking up some knowledge. Bit harder than "good first issues"For beginners in QC, this will help picking up some knowledge. Bit harder than "good first issues"good part-time projectA meaty non-urgent issue with a substantial amount of work to be done.A meaty non-urgent issue with a substantial amount of work to be done.kind/design-issueA conversation around designA conversation around designtriage/acceptedA consensus emerged that this bug report, feature request, or other action should be worked onA consensus emerged that this bug report, feature request, or other action should be worked on