Skip to content

Commit

Permalink
Switch QIR Target checking from allow list to deny list
Browse files Browse the repository at this point in the history
This change updates the logic we use to decide if a target supports QIR submission in the extension. Previously it was an allow list which required any new targets or providers to be explicitly added. Instead, this uses a deny list style so that new providers and targets are allowed to submit either base or adaptive without requiring updates to the QDK. Then we can coordinate with new partners on what targets should be lit up for which purposes.
  • Loading branch information
swernli committed Dec 9, 2024
1 parent f026774 commit c47bb8f
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions vscode/src/azure/providerProperties.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,8 @@ export function targetSupportQir(target: string) {
// Note: Most of these should be dynamic at some point, with configuration coming
// from the service, and able to be overridden by settings.
return (
target.startsWith("ionq") ||
target.startsWith("quantinuum") ||
target.startsWith("rigetti")
!(target == "microsoft.estimator") &&
!(target.startsWith("microsoft") && target.endsWith("cpu"))
);
}

Expand All @@ -28,5 +27,5 @@ export function shouldExcludeProvider(provider: string) {
}

export function supportsAdaptive(target: string) {
return target.startsWith("quantinuum");
return !target.startsWith("ionq") && !target.startsWith("rigetti");
}

0 comments on commit c47bb8f

Please sign in to comment.