Skip to content
This repository was archived by the owner on Aug 21, 2023. It is now read-only.

Resolved deprecation warnings in transpiler tutorial notebook#999

Merged
nonhermitian merged 6 commits into
Qiskit:masterfrom
wagnersj:update-transpilertutorial
Sep 21, 2020
Merged

Resolved deprecation warnings in transpiler tutorial notebook#999
nonhermitian merged 6 commits into
Qiskit:masterfrom
wagnersj:update-transpilertutorial

Conversation

@wagnersj
Copy link
Copy Markdown
Contributor

@wagnersj wagnersj commented Aug 10, 2020

Summary

Resolved deprecation warnings in issue #990 related to changes in qiskit-terra v0.14.0 were appearing in the transpiler tutorial notebook.

Details and comments

Resolved qiskit.extensions.standard library deprecations by using qiskit.circuit.library.standard_gates library instead.

Resolved deprecated function calls in 'DAGCircuitandLayoutclasses from theBasicSwapclass defined in the notebook. Basically, a cut-and-paste from theqiskit.transpiler.passes.BasicSwap`.

Also added some helpful documentation in the Introduction section about installing pydot and graphviz, which are needed for dag_drawer.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Aug 10, 2020

CLA assistant check
All committers have signed the CLA.

@wagnersj wagnersj closed this Aug 12, 2020
@wagnersj wagnersj reopened this Aug 13, 2020
@wagnersj
Copy link
Copy Markdown
Contributor Author

Closed wrong PR

nonhermitian
nonhermitian previously approved these changes Sep 7, 2020
Copy link
Copy Markdown
Contributor

@nonhermitian nonhermitian left a comment

Choose a reason for hiding this comment

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

Many thanks for this!

@nonhermitian
Copy link
Copy Markdown
Contributor

Actually just need the conflict merged a ready to go.

Conflicts:
	tutorials/circuits_advanced/4_transpiler_passes_and_passmanager.ipynb
@nonhermitian nonhermitian merged commit e60c750 into Qiskit:master Sep 21, 2020
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.

3 participants