Skip to content
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

Ensure BQSKit example runs #1962

Merged
merged 3 commits into from
Aug 16, 2023
Merged

Ensure BQSKit example runs #1962

merged 3 commits into from
Aug 16, 2023

Conversation

natestemen
Copy link
Member

@natestemen natestemen commented Aug 14, 2023

Background

The BQSKit tutorial was originally added in #1489, and after a few modifications to get it running on the 1.0 release of BQSKit, it was running smoothly. In mid-July the import bqskit line started started complaining, giving the following AttributeError.

AttributeError: module 'pyximport.pyximport' has no attribute 'get_distutils_extension'

Andrea hot-fixed this by skipping code execution on the example in #1927.

Description

In working on this issue, I was not able to reproduce the import issue locally. I wonder if it is a Linux only issue. This PR resolves this issue by bumping the BQSKit version to the latest version (1.0.4).

fixes #1920

@github-actions
Copy link

Binder 👈 Launch a binder notebook on branch unitaryfund/mitiq/nts-bqskit-restart

@codecov
Copy link

codecov bot commented Aug 14, 2023

Codecov Report

Merging #1962 (ee3a807) into master (e9ebb86) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1962   +/-   ##
=======================================
  Coverage   98.36%   98.36%           
=======================================
  Files          83       83           
  Lines        3982     3982           
=======================================
  Hits         3917     3917           
  Misses         65       65           

@natestemen natestemen self-assigned this Aug 15, 2023
@natestemen natestemen added documentation Improvements or additions to documentation tech debt Technical Debt that should be paid off! interface-and-conversions How Mitiq interfaces with quantum software packages. labels Aug 15, 2023
Copy link
Member

@andreamari andreamari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for closing many tech debts!

@andreamari andreamari merged commit bd596b4 into master Aug 16, 2023
20 checks passed
@natestemen natestemen deleted the nts-bqskit-restart branch October 24, 2023 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation interface-and-conversions How Mitiq interfaces with quantum software packages. tech debt Technical Debt that should be paid off!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bqskit example in docs is failing
2 participants