Skip to content
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

[OpenPulse] PyDataMem_NEW_ZEROED is not declared #578

Closed
doichanj opened this issue Feb 4, 2020 · 1 comment
Closed

[OpenPulse] PyDataMem_NEW_ZEROED is not declared #578

doichanj opened this issue Feb 4, 2020 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@doichanj
Copy link
Collaborator

doichanj commented Feb 4, 2020

Informations

  • Qiskit Aer version: 0.4
  • Python version: 3.7
  • Operating system: Red Hat Enterprise Linux Server release 7.6 (ppc64)

What is the current behavior?

Compilation failure occurs on (only?) Power + RedHat environment because of Numpy's function "PyDataMem_NEW_ZEROED", maybe it is not implemented on this environment.
The error message is as follow (same 7 errors occur while compiling OpenPulse)

/gpfs/wscgpfs01/doichan/Q/test/qiskit-aer/_skbuild/linux-ppc64le-3.7/cmake-build/qiskit/providers/aer/pulse/qutip_lite/cy/spmath.cxx:3677:52: error: ‘PyDataMem_NEW_ZEROED’ was not declared in this scope __pyx_v_mat->data = ((__pyx_t_double_complex *)PyDataMem_NEW_ZEROED(__pyx_v_nnz, (sizeof(__pyx_t_double_complex)))); ^~~~~~~~~~~~~~~~~~~~ /gpfs/wscgpfs01/doichan/Q/test/qiskit-aer/_skbuild/linux-ppc64le-3.7/cmake-build/qiskit/providers/aer/pulse/qutip_lite/cy/spmath.cxx:3677:52: note: suggested alternative: ‘PyDataMem_NEW’ __pyx_v_mat->data = ((__pyx_t_double_complex *)PyDataMem_NEW_ZEROED(__pyx_v_nnz, (sizeof(__pyx_t_double_complex)))); ^~~~~~~~~~~~~~~~~~~~ PyDataMem_NEW

Steps to reproduce the problem

What is the expected behavior?

Suggested solutions

Could you please replace PyDataMem_NEW_ZEROED to PyDataMem_NEW that maybe supported on the Power + RedHat environment.
I googled PyDataMem_NEW_ZEROED, but it seems very few people uses this function, so I think using PyDataMem_NEW is a common way.

@doichanj doichanj added the bug Something isn't working label Feb 4, 2020
@nonhermitian
Copy link
Contributor

One has to be a bit careful as some routines might be expecting a zeroed memory.

doichanj added a commit to doichanj/qiskit-aer that referenced this issue Feb 19, 2020
@atilag atilag closed this as completed in 0fe86a7 Feb 20, 2020
mtreinish pushed a commit to mtreinish/qiskit-aer that referenced this issue Feb 24, 2020
mtreinish pushed a commit to mtreinish/qiskit-aer that referenced this issue Feb 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants