-
Notifications
You must be signed in to change notification settings - Fork 354
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
Issue #32: Documentation fixes in CONTRIBUTING and fix import in terra-integrated tests #33
Conversation
… the command) -—install => --install
if you had to use --plat-name macosx-10.9-x86_64 for bdist_wheel then you need to do this for install: python ./setup.py install -- -DCMAKE_OSX_DEPLOYMENT_TARGET:STRING=10.9 -DCMAKE_OSX_ARCHITECTURES:STRING=x86_64
Fixed "from qiskit.quantum_info import state_fidelity"
@@ -257,6 +257,8 @@ and then run `unittest` Python framework. | |||
.. code:: | |||
|
|||
qiskit-aer$ python ./setup.py install | |||
# if you had to use --plat-name macosx-10.9-x86_64 for bdist_wheel then you need to do this for install: | |||
# python ./setup.py install -- -DCMAKE_OSX_DEPLOYMENT_TARGET:STRING=10.9 -DCMAKE_OSX_ARCHITECTURES:STRING=x86_64 |
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.
Can't we use --plat-name macosx-10.9-x86_64
here as well?
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.
Ok, we can`t.
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.
--plat-name is an option for the bdist_wheel
subcommand and not others like install
@atilag have you ever seen the following json-related error when trying to build aer, more specifically, when Building CXX object CMakeFiles/qasm_simulator.dir/contrib/standalone/qasm_simulator.cpp.o:
|
I also haven't been able to figure out how to build & install aer using I'll try other incantations... |
Issue Qiskit#32: Documentation fixes in CONTRIBUTING and fix import in terra-integrated tests
Summary
Fixed a couple documentation errors I encountered when building.
Details and comments
Fixed a bad dash character and added cmake flags for installing the terra-integrated tests.