Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@bamarsha
Copy link
Contributor

@bamarsha bamarsha commented Apr 7, 2021

No description provided.

Chris Granade and others added 29 commits March 16, 2021 23:42
* Implement quantumlibraries#413.

* Add missing open statement.

* Fix build warnings.

* Addressing feedback.
* Turn on QirRuntime build/validation by default

* Update package versions, remove redundant main

* install prereqs by default too

* Correctly throw when QIR build/test fails.

* Update tests for latest QIR compiler features

* Dont gate QIR tests on fullstate simulator flag

* Use solution build to generate QIR
* Changed quantum__rt__heap_alloc() signature
* Added quantum__rt_string_get_(data|length)
* Changed the __quantum__rt__*_update_reference_count signature
* Changed the __quantum__rt__string_create signature
* Added to quantum__rt__qubit_release_array() the array release
* Changed the __quantum__rt__.*_update_alias_count() signature
* Added __quantum__rt__result_get_zero and a __quantum__rt__result_get_one
* Added __quantum__rt__capture_update_reference_count and __quantum__rt__capture_update_alias_count
* Added __quantum__rt__memory_allocate()
* Update QIR scripts to latest compiler support
Co-authored-by: Stefan J. Wernli <swernli@microsoft.com>
This fixes an issue where the fallback logic in the `QCTraceSimulator` for dynamically loading the QSharp.Core target package wouldn't work for manually written C# drivers. This fixes the issue by introducing new fallback logic that relies on the assembly name of the current simulator assembly to determine the full name of the QSharp.Core assembly to load.

Fixes #540
Fixes #542
If inheritance is used, the `this` in QCTraceSimulatorImpl wont resolve to the right assembly. This uses the QCTraceSimulatorImpl type itself so that the assembly matching the QDK can be resolved.
* Turned off the test for Message()
* Moved lib/QSharpFoundation/intrinsicsOut.cpp -> lib/QIR/rtOut.cpp.
* Moved SetOutputStream()
* Build fix.
* Uncommented the test
* Clean-up.
* CR changes
Added Multiple Entry points for the Entry Point Driver
#580)

* Update C++ entry-point reference driver to use latest QIR emission.
* Updated test script.
* Added --string-array option.
* Addressed Robin's feedback.
* Update expectation for Range array.
* Free vector of allocated pointers.
* Update test.py.
* Addressed additional feedback from Robin.
* Added DrawRandomDouble
* Fixed #565
* Add Adj + Ctl to Assert methods that support it

Fixes Add Adj functor to AssertQubit #316

* Don't propagate Ctl we don't want to support long term

* Fix spacing

* Update doc comments and AssertQubit callables
* Remove BitStates.hpp

BitStates.hpp defined a custom implementation for tracking vectors of bits. However, `vector<bool>` from the STL already is a specialization of `vector` designed to save space and use a single bit for each index. Since we don't want to support the custom type in our API nor do we really need one internally, it was removed and all instances replaced with `vector<bool>`.

Fixes #577

* CR feedback
* fix links

* revert unwanted changes
* Updated the build/test instructions in README.md
* Updated the root-level README.md
* Move files.
* Rename files.
* Remove projects from Simulation solution.
* Update scripts.
* Add projects to simulation.
* Added debugging couts for build pipeline failure.
* Fix relative path for simulator dll discovery
* Apply path fix to samples too
* Revert "Added debugging couts for build pipeline failure."
This reverts commit e51b38e.
Co-authored-by: Stefan J. Wernli <swernli@microsoft.com>
* Minor build script clean-up

* Update gitignore for PR comments

* Update src/Qir/Runtime/build-qir-runtime.ps1

Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>

Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
This change separates the QIR Runtime (production code) from the tests and samples. Since the Runtime has no Q# dependencies, we can now pull it into the native build step of the e2e pipeline to better produce cross-platform binaries from the builds.
@bamarsha bamarsha force-pushed the samarsha/qep2-main branch from 759a61a to b9c49cc Compare April 7, 2021 17:48
@bamarsha bamarsha marked this pull request as ready for review April 8, 2021 02:34
@bamarsha bamarsha requested a review from swernli April 8, 2021 02:34
@bamarsha bamarsha merged commit da2190a into feature/qep2 Apr 8, 2021
@bamarsha bamarsha deleted the samarsha/qep2-main branch April 8, 2021 03:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants