This repository was archived by the owner on Jan 12, 2024. It is now read-only.
Integrate ExecutionPathTracer in IQ# #231
Merged
Conversation
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
* Add ExecutionPathTracer to repo * Add WithExecutionPathTracer extension method to SimulatorBase * Respond to PR feedback * Use GetRuntimeMetadata * Remove ApplyToEach * Add tests * Use Newtonsoft instead of System.Text.Json * Rename ArgStr to DisplayArgs * Clean up code * Fix typo in tsconfig * Fix tests
* Add view magic command * Rename ExecutionPathEncoder to DisplayableHtmlEncoder * Add documentation to view command * Rename %view to %trace * Fix JSON parsing * Add TraceMagic test * Add flag for depth * Fix nested test * Empty commit to trigger CI * Empty commit to trigger CI
* Change depth flag to --depth * Add parentheses around DisplayArgs if it doesn't have it * Trigger CI rebuild
* Add tests for operations with operations as arguments * Build 0.12.2007.2301. Co-authored-by: Project Collection Build Service (ms-quantum) <>
* Implement isMeasurement flag for execution path operations * Rename 'controlled' and 'adjoint' to 'isControlled' and 'isAdjoint'
* Allow default depth for %trace to be set in %config * Fix config magic description
theRoughCode
force-pushed
the
feature/path_visualizer
branch
from
add8675 to
64fdbd2
Compare
theRoughCode
requested review from
anpaz,
cgranade,
ricardo-espinoza and
rmshaffer
and removed request for
anpaz and
cgranade
|
I'm making this PR open for review because #219 is blocked on me figuring out how to write tests involving |
rmshaffer
previously approved these changes
Jul 30, 2020
|
One thing that just occurred to me: If you want syntax highlighting for iqsharp/src/Kernel/client/kernel.ts Line 69 in b69beea (I also notice that the new-ish command |
|
@rmshaffer opened a PR for that at #240 :) |
* Add composite operation handling * Add ForEach tests * Add Canon.qs to Tests.IQSharp.csproj * Build 0.12.2007.2412. * Fix ResetAll tests * Fix tests to allow using Microsoft.Quantum.Canon * Add newline * Build 0.12.2007.3008. * Reformat ExecutionPathTracer stylistically * Fix iqsharp.sln * Empty commit to retrigger CI * Remove dotnet test args * Fix broken build * Fix TraceMagic test * Fix broken build tests Co-authored-by: Project Collection Build Service (ms-quantum) <>
* Add syntax highlighting for %trace command * Fix ordering and add lsopen
rmshaffer
approved these changes
Jul 31, 2020
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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 addresses #158 and merges the
feature/path_visualizerbranch intomaster. This PR contains the following implementations:ExecutionPathVisualizerJS/TS code + Jest testsExecutionPathTracerC# code that hooks on to a simulator and traces out incoming operations and theirRuntimeMetadatato generate the correspondingExecutionPathJSON string%tracemagic command to trigger this visualization process--depthflag for%tracecommand that allows users to select the render depth of operations1) in%configDisplayableHtmlElementandDisplayableHtmlElementEncoderto render HTML elements to Jupyter cellsNote: Waiting on #219 to get merged in to this branch as well, but facing issues with writing tests that involve
Microsoft.Quantum.Canon.