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 a more recent SUNDIALS version #113

Open
sanjayankur31 opened this Issue Nov 10, 2018 · 5 comments

Comments

Projects
None yet
2 participants
@sanjayankur31
Copy link

sanjayankur31 commented Nov 10, 2018

The copy of SUNDIALS that is bundled with neuron is quite old, old enough that the API for SUNDIALS seems to have changed since. It'll be good if neuron could be updated to use the latest SUNDIALS code, to take advantage of its optimisations and bug fixes. This will also make it much easier for downstream distributions to provide neuron.

The current version of SUNDIALS is 3.2.1: https://computation.llnl.gov/projects/sundials/sundials-software

@brunomaga

This comment has been minimized.

Copy link

brunomaga commented Nov 10, 2018

I just had the same conversation with @nrnhines . I managed to update CVODES to version 3.2.1 on a subset of NEURON code, and will start working on porting it fully to NEURON on the 19th November. Hopefully by the end of the month will be completed.

@sanjayankur31

This comment has been minimized.

Copy link

sanjayankur31 commented Nov 10, 2018

Ah, thank you very much. I was about to embark on it myself, but given my lack of experience with sundials, it'd be quite the task. Would you have a fork where you're working on this that I can watch for updates?

@brunomaga

This comment has been minimized.

Copy link

brunomaga commented Nov 10, 2018

I will create a branch when I start the development on the 19th, and let you know.

@brunomaga

This comment has been minimized.

Copy link

brunomaga commented Nov 23, 2018

You can follow the development in this fork:
https://github.com/brunomaga/nrn
Beta version is done, we are now in testing stage. @nrnhines will then decide about the merge.

@sanjayankur31

This comment has been minimized.

Copy link

sanjayankur31 commented Nov 24, 2018

Thanks very much. I'll test out and give any feedback if it comes up.

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