-
Notifications
You must be signed in to change notification settings - Fork 5
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
fix: #521: exclude isa dead qubits from quilc ISA #522
base: main
Are you sure you want to change the base?
Conversation
7a1d57a
to
3540e36
Compare
|
3540e36
to
12b4ee5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good so far - please address that comment and add test coverage for it then we're 👍
crates/lib/src/compiler/isa/mod.rs
Outdated
let edges = edges | ||
.into_iter() | ||
.map(|(k, v)| (k.to_string(), v)) | ||
.filter(|(_, e)| e.has_valid_operations()) | ||
.collect(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To get ahead of a related error - should only be those edges for which both qubits are also included
9f59ab3
to
d7d26e0
Compare
Resolves #521