-
Notifications
You must be signed in to change notification settings - Fork 584
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
Add cuQuantum-based TFQ native ops and layers. #770
Conversation
Upgrade bazel version 5.3.0 and fix some typo in tf version (tensorflow#755)
Custatevec support v5
Fix BulkSetAmpl bug, Add more diff cuquantum unit tests
update configure
Benchmark tests for cuquantum ops
Disable use_cuquantum with cirq backend
Update benchmark code
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
@@ -11,7 +11,7 @@ | |||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | |||
# See the License for the specific language governing permissions and | |||
# limitations under the License. | |||
# ============================================================================== |
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.
Would you be able to revert the files where there are little to no changes made to the actual code so that the diff view is easier to read ?
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.
Sure, let me update the PR soon :) thank you Michael.
Breaking Changes
Major Features and Improvements
Source: https://github.com/Sinestro38/quantum/blob/master/tensorflow_quantum/release.md |
Hi I tried all the above and when ran the tests, these failed:
the output of the second test log being
|
|
iI followed those, I didn't get any error during the build instructions. when running configure I do get the message
I tried again and see this error in one of the tests
I see that I have when running
which I do |
managed to make it work (though is not using gpus significantly) but that's for later. Thank you! |
This has large changes and requires some deps error, rewrite documentations, and fixed internal bugs inside random ops, and so forth.
Please look at the PR descriptions in my repo to understand the procedures : https://github.com/jaeyoo/quantum/pulls?q=is%3Apr+is%3Aclosed