-
Notifications
You must be signed in to change notification settings - Fork 21
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
Testing of Terra Upgrade Branch #10
Comments
Installation Issues
To avoid this error following file
|
That is possibly because either you have not installed Rust, or Rust is not in your PATH. Please check that. |
Is Rust a dependency for the updated Terra? I do remember Qiskit moving towards |
My bad, I've installed rust but it was not in the path. It works in my desktop but not in Colab etc. (even after installing rustc). |
Yes, Rust is a dependency for the updated Terra. I am unsure as to how one would make this work in Colab |
I did some basic test and the code works so far. I want to do a few more tests. There should be a way to make this work in Colab or Binder. I will look for it meanwhile if someone find a way leave a comment here. C-DAC uses the master branch in their supercomputer(at least that's what I am told). I will give them a heads up before going through the process of merging this to the master branch. |
Please do. Thanks a lot! |
@biplab37 While giving a heads-up to the people at CDAC, please do not forget to mention that the output format has changed ever so slightly. Now it is an object, not a dictionary |
I think the output has always been a |
Now the output is a Result object with the Result.results being an ExperimentalData object |
If anyone else did not find any obvious mistake then we can merge the |
So, there are some issues that we see which we are trying to fix right now. Once they are done we would let you know |
Hi all. At this point, the terra_upgrade branch is functional and runs all examples with the expected output and passes all tests. I would like to know if you have any other tests that you would like to perform before we can merge this into the main repository.
The text was updated successfully, but these errors were encountered: