-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Conversation
should be done but I will run it once more in the morning. |
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.
We should not be using Qconfig files in this notebook. We do not need them, and are trying to kill them. We should just say token and, if in the IBM Q network, a url. If we want to highlight how to use the old Qconfig, then it should be done later in the notebook.
Also, you killed the job status because it took too long, and this is shown in the notebook. Why not use the Jupyter status monitor?
The qconfig here is not a old style qconfig it’s just a file that holds the url and token as I don’t want to display them. Read the flow and you see why. |
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.
This is well done.
Thanks @ajavadia next is visualizations |
@ajavadia your fix broke. |
until its fixed from your changes
thanks @ajavadia it works. Please approve again and @nonhermitian have a look. |
Finishing getting started IBMQ provider This is part of issue #293