This repository was archived by the owner on Jan 12, 2024. It is now read-only.
Add support for target packages (merge from feature/decomp) - #476
Merged
Stefan J. Wernli (swernli) merged 80 commits intoJan 29, 2021
Merged
Add support for target packages (merge from feature/decomp)#476Stefan J. Wernli (swernli) merged 80 commits into
Stefan J. Wernli (swernli) merged 80 commits into
Conversation
Copy link
Copy Markdown
Collaborator
Sorry, something went wrong.
* Refactor QSharp.Core into Foundation + Core * Trying to fix sln * Splitting Utils.qs * Moving ClassicalControl.qs * Reverting Intrinsic.I change
Merging master and fixing up solution file.
Merge remote-tracking branch 'origin/master' into swernli/decomp-merge-master
Merge remote-tracking branch 'origin/master' into feature/decomp
Merge remote-tracking branch 'origin/master' into feature/decomp
Something about the merge from master makes it so the new tests for Honeywell and QCI executables fail when run against the resource estimator. By skipping these tests temporarily, I can at least be unblocked to continue development.
This change splits all the intrinsics and decompositions defined in QsharpCore out into their own files to allow them to be individually combined into a target definition package. Q# should include decompositions to alternate quantum gate sets #249
…tiQubit.qs Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
Create QsharpCore targeting package
This changes continues building on the work in feature/decomp to enable targeting alternate gate sets via packages specifying the supported set of intrinsics and decompositions for that gate set. This introduces Type2, which targets a set of intrinsics that includes SWAP and the 3 Pauli Ising gates. It also adds support for executing against this target package in the QDK simulator, and shares some of the existing intrinsic unitary tests to validate behavior of the package. See #249 for the rundown on what work is part of this overall feature.
Merge master into feature/decomp
Fixing csproj QDK versions
This change adds the Type1 target package for Q# targets that support a limited number of control operations. The change includes support for simulating a Type1 package and tests that verify unitary gate behavior for that package. This part of the ongoing work described in #249. Co-authored-by: Chris Granade <chgranad@microsoft.com> Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
Merging main to feature/decomp
Copy link
Copy Markdown
|
Azure Pipelines successfully started running 2 pipeline(s). |
Sorry, something went wrong.
Stefan J. Wernli (swernli)
added a commit
to microsoft/QuantumLibraries
that referenced
this pull request
Jan 21, 2021
With the ongoing refactor of the qsharp-runtime repo to support target packages, the Microsoft.Quantum.Simulators package no longer automatically brings in the Microsoft.Quantum.QSharp.Core package, so projects need to explicitly add that reference if they are have suppressed loading of the core package via `IncludeQsharpCorePackages` set to false. See microsoft/qsharp-runtime#476, which this change will help unblock.
Stefan J. Wernli (swernli)
added a commit
to microsoft/iqsharp
that referenced
this pull request
Jan 22, 2021
With the ongoing refactor of the qsharp-runtime repo to support target packages, the Microsoft.Quantum.Simulators package no longer automatically brings in the Microsoft.Quantum.QSharp.Core package, so projects need to explicitly add that reference if they are not using the QDK. See microsoft/qsharp-runtime#476, which this change will help unblock.
Stefan J. Wernli (swernli)
added a commit
to microsoft/QuantumLibraries
that referenced
this pull request
Jan 22, 2021
With the ongoing refactor of the qsharp-runtime repo to support target packages, the Microsoft.Quantum.Simulators package no longer automatically brings in the Microsoft.Quantum.QSharp.Core package, so projects need to explicitly add that reference if they are have suppressed loading of the core package via `IncludeQsharpCorePackages` set to false. See microsoft/qsharp-runtime#476, which this change will help unblock.
Stefan J. Wernli (swernli)
added a commit
to microsoft/Quantum
that referenced
this pull request
Jan 22, 2021
* Refactor QSharp.Core into Foundation + Core * Trying to fix sln * Splitting Utils.qs * Moving ClassicalControl.qs * Reverting Intrinsic.I change
Bettina Heim (bettinaheim)
approved these changes
Jan 28, 2021
Bettina Heim (bettinaheim)
left a comment
Copy link
Copy Markdown
Contributor
There was a problem hiding this comment.
Approving as per our code review meeting on behalf of the team.
(Please look into the perf tests before merging. Thanks!)
Sorry, something went wrong.
Copy link
Copy Markdown
Collaborator
Author
|
/azp run |
Sorry, something went wrong.
Copy link
Copy Markdown
|
Azure Pipelines successfully started running 2 pipeline(s). |
Sorry, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Reviewers
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR merges the payload from feature/decomp to enable target packages for Q# runtime. This resolves #249 (see that issue for details on the individual elements and links to previous PRs and discussions). Here's the high level summary of the contents: