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

Add releases folder with Terra 0.6 and 0.7 release guides.#447

Merged
jaygambetta merged 11 commits intoQiskit:masterfrom
dongreenberg:terra0_7
Dec 26, 2018
Merged

Add releases folder with Terra 0.6 and 0.7 release guides.#447
jaygambetta merged 11 commits intoQiskit:masterfrom
dongreenberg:terra0_7

Conversation

@dongreenberg
Copy link
Copy Markdown
Contributor

Introduces a "releases" folder for notebook guides to major Qiskit releases, including guides for Terra 0.7 and 0.6 (Ali agreed this made sense). I am working on the guide for Aqua 0.4 presently.

(Third time's a charm 😁)

@jaygambetta
Copy link
Copy Markdown
Member

you need to rerun and you have some errors. I see hex counts as well.

@jaygambetta
Copy link
Copy Markdown
Member

Also now we have updated to master please push there

Copy link
Copy Markdown
Member

@jaygambetta jaygambetta left a comment

Choose a reason for hiding this comment

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

See above

@mtreinish mtreinish changed the base branch from terra0_7 to master December 19, 2018 20:07
@mtreinish
Copy link
Copy Markdown
Member

sigh, I apologize for that. Github clearly did the wrong thing when I adjusted the base from the terra branch to master. It looks like it's trying to remerge the old state of master from when things branched. I have no idea how github could screw up a simple rebase. I'll fix this locally and force push

@mtreinish mtreinish changed the base branch from master to terra0_7 December 19, 2018 20:10
@mtreinish mtreinish changed the base branch from terra0_7 to master December 19, 2018 20:14
@mtreinish
Copy link
Copy Markdown
Member

OK, I've manually rebased this onto master (the issue was that terra0_7 and master don't rebase cleanly anymore so there are unresolvable merge conflicts elsewhere in the history unrelated to this patch) so I just bypassed that and targeted the commit on master for you. @dongreenberg you will still need to rerun it though per @jaygambetta review earlier. I only fixed the git side of things.

@attp
Copy link
Copy Markdown
Member

attp commented Dec 20, 2018

@dongreenberg: from qiskit.transpiler.passes import BasicMapper, CXCancellation, LookaheadMapper causes an error: ImportError: cannot import name 'BasicMapper' from 'qiskit.transpiler.passes'

@dongreenberg
Copy link
Copy Markdown
Contributor Author

  • Notebook corrected and runs.
  • The hex is WAI, part of demonstrating to users why they should not use results.data().

@dongreenberg dongreenberg reopened this Dec 20, 2018
@jaygambetta
Copy link
Copy Markdown
Member

jaygambetta commented Dec 20, 2018

These look good

In terra 0.6 can you remove the interacting plotting or whatever is making the red text of errors.

In terra 0.7

  • don't use the legacy i would just use aer and basic aer.
  • the stocastic swap is there for the mapper
  • also update the backends

@dongreenberg
Copy link
Copy Markdown
Contributor Author

Thanks, didn't see that. Cleaned everything up so both files look presentable in github.

@jaygambetta
Copy link
Copy Markdown
Member

jaygambetta commented Dec 21, 2018

Can you switch from mapper to StochasticSwap. This is what meant above

@jaygambetta
Copy link
Copy Markdown
Member

Also, there is still not anything about the backend updates.

@jaygambetta
Copy link
Copy Markdown
Member

I also see that you have LegacySimulators this should Aer

@jaygambetta
Copy link
Copy Markdown
Member

Can you also make the location just releases not qiskit/resleases

@jaygambetta jaygambetta merged commit dccb9b8 into Qiskit:master Dec 26, 2018
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.

4 participants