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

Using NESTML with repo-based NEST and Boost #542

Closed
Silmathoron opened this issue Mar 17, 2020 · 10 comments · Fixed by #691
Closed

Using NESTML with repo-based NEST and Boost #542

Silmathoron opened this issue Mar 17, 2020 · 10 comments · Fixed by #691
Assignees

Comments

@Silmathoron
Copy link
Member

Silmathoron commented Mar 17, 2020

We now have a bunch of pre-built NEST from repo (see install).
However, at least for the ubuntu package, the source files are not included in a way that is automatically detected by the NESTML install script (though for ubuntu it is /usr/include/nest).
Since these installation methods are likely to become dominant, I think we need to address this issue.
Similar problem arises if NEST is compiled with -Dwith-boost=1 as the include path is not contained automatically; this option will become default in NEST3.

Steps to reproduce: build a model with default NESTML config with a prebuilt NEST or with a NEST built with Boost.

Potential link to #480 and #503

@jougs
Copy link
Contributor

jougs commented Apr 9, 2020

@clinssen: Can this issue be solved in NESTML directly, or should it rather be addressed by changes to the nest-config script? Thanks!

@clinssen
Copy link
Contributor

clinssen commented Apr 9, 2020

@Silmathoron: sorry for the late reply. Could you report the output of nest-config --includes?

@Silmathoron
Copy link
Member Author

Not right now, unfortunately, I'm not a Debian user, but I'll try to do it soon.
Is Boost supposed to be in there too (do I need to provide it for the self-compiled install with boost)?

@clinssen
Copy link
Contributor

clinssen commented Apr 9, 2020

I believe NEST is receiving some updates right now related to Boost. Please hold tight until nest/nest-simulator#1502 and nest/nest-simulator#1477 are merged.

@tfardet
Copy link

tfardet commented Sep 16, 2021

Hey there, additional question regarding the use of NESTML with deb-repo NEST: it seems that install_nest would typically then try to install to the system location after calling the NEST config files, which of course leads to a permission error.
Is there currently a way to address this or a plan to fix it in the near future or does NESTML only work with local builds?
Asking because I'm getting questions from people that used the Ubuntu PPA and now try to use NESTML.

EDIT: could this be a temporary workaround?

@jougs jougs self-assigned this Sep 16, 2021
@jougs
Copy link
Contributor

jougs commented Sep 16, 2021

Dear @tfardet, I am not fully convinced by the workaround and think the proper solution is to actually support the CMake switch -DCMAKE_INSTALL_PREFIX in the NESTML generated NEST extension module.

Currently, we just overwrite the prefix by whatever we get from nest-config. If we would just check if the user set a different location and use that value instead, users could install to arbitrary locations and then set LD_LIBRARY_PATH before running NEST.

I'm not yet completely sure how to handle install_nest in this situation, but think this should also be possible without too much hassle.

@tfardet
Copy link

tfardet commented Sep 16, 2021

Hi @jougs yeah, actually I tried the potential workaround but it does not work anyways, and it's definitely not a long-term solution.
Since I don't think there's any reasonable way to install to /usr, I would propose to default to $HOME/.local if NEST has been installed system-wide with an optional parameter to override this if the user wants.
Maybe a note somewhere warning users about using NESTML with system-wide NEST installs in the meantime?

@jougs
Copy link
Contributor

jougs commented Sep 17, 2021

I'm working on this today and am somewhat confident to have a fix by tonight.

@tfardet
Copy link

tfardet commented Oct 16, 2021

Hi @jougs, any news on the prefix front?

@jougs
Copy link
Contributor

jougs commented Oct 26, 2021

Work is now happening in #691. Sorry for the delay.

@jougs jougs linked a pull request Dec 2, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants