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

Does NEST need to link against libneurosim or libpyneurosim? #1063

Closed
sanjayankur31 opened this Issue Oct 28, 2018 · 5 comments

Comments

Projects
None yet
4 participants
@sanjayankur31
Copy link
Contributor

sanjayankur31 commented Oct 28, 2018

Hello,

At buildtime, does NEST need to link against libneurosim or libpyneurosim or both? Currently, it expects them both in the same directory, /usr/lib{,64}, but at runtime, it seems to require libpyneurosim in PYTHONPATH which is not /usr/lib{,64} (it should be /usr/lib{,64}/pythonX.Y/site-packages).

I had opened this PR against libneurosim where I'd moved libpyneurosim to the correct python path, but now I'm not sure if that's correct anymore: INCF/libneurosim#12

@heplesser

This comment has been minimized.

Copy link
Contributor

heplesser commented Oct 28, 2018

@jougs Could you answer this one?

@sanjayankur31

This comment has been minimized.

Copy link
Contributor

sanjayankur31 commented Nov 20, 2018

@jougs would you have any comments here please?

@jougs

This comment has been minimized.

Copy link
Contributor

jougs commented Nov 21, 2018

@mdjurfeldt: can you please comment on this one?

@mdjurfeldt

This comment has been minimized.

Copy link
Contributor

mdjurfeldt commented Nov 23, 2018

I've answered this in the libneurosim issue, but copy the text here:

"libpyneurosim is not a Python extension. It is rather support for using connection generators written in Python from C++ code (i.e. the reverse of what Python extensions do). Therefore, libpyneurosim should be installed in ordinary library directories like /usr/lib{,64}. NEST needs to link against both libpyneurosim and libneurosim and these libraries need to be found at link time by the NEST binary.

Also, this means that the comment in the NEST example is wrong. These libraries should not be on PYTHONPATH but on LD_LIBRARY_PATH (if needed).

I've commited an explanation in the libneurosim README.md."

@mdjurfeldt

This comment has been minimized.

Copy link
Contributor

mdjurfeldt commented Nov 23, 2018

sanjayankur31 added a commit to sanjayankur31/nest-simulator that referenced this issue Nov 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment