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

Fix pillow import in optionals #7774

Merged
merged 5 commits into from
Mar 14, 2022
Merged

Conversation

enavarro51
Copy link
Contributor

Summary

Fixes #7767

Details and comments

This fixes the pillow import using the suggestion from @jakelishman in #7767 that was causing some of the drawers to fail.

@enavarro51 enavarro51 requested review from a team, manoelmarques and woodsp-ibm as code owners March 14, 2022 16:21
Copy link
Member

@mtreinish mtreinish left a comment

Choose a reason for hiding this comment

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

Overall LGTM, but we can drop the release note

releasenotes/notes/fix-pillow-import-95bf52acd75fbb20.yaml Outdated Show resolved Hide resolved
@mtreinish mtreinish added the Changelog: None Do not include in changelog label Mar 14, 2022
@mtreinish mtreinish added this to the 0.20 milestone Mar 14, 2022
@coveralls
Copy link

Pull Request Test Coverage Report for Build 1981848336

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 83.463%

Totals Coverage Status
Change from base Build 1981749032: 0.0%
Covered Lines: 52331
Relevant Lines: 62700

💛 - Coveralls

@mergify mergify bot merged commit 1980326 into Qiskit:main Mar 14, 2022
ElePT pushed a commit to ElePT/qiskit-algorithms that referenced this pull request Jul 27, 2023
* Change optionals import for pillow

* Text change and co-author

Co-authored-by: Jake Lishman <[email protected]>

* Remove release note

Co-authored-by: Jake Lishman <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: None Do not include in changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MissingOptionalLibraryError for pillow
3 participants