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

Allow Banded Jacobian Matrix #113

Closed
YingboMa opened this issue Jul 29, 2017 · 5 comments
Closed

Allow Banded Jacobian Matrix #113

YingboMa opened this issue Jul 29, 2017 · 5 comments

Comments

@YingboMa
Copy link

It will be really nice to add banded Jacobian matrix support in NLsolve.jl.

@ChrisRackauckas
Copy link
Contributor

I think this would be mostly solved by

#83

since the user can just set it to use a banded linear solver. That doesn't fix the memory usage though, so it would still be nice to allow the user to give a type which is the suggested Jacobian type. This would be a nice way IMO to handle sparsity and other things like Triangular Jacobians too.

@YingboMa
Copy link
Author

YingboMa commented Aug 8, 2017

I suggest that there should be a functionality like nlsolve(f!, u0, jac_upper=N, jac_lower=M), so user doesn't need to calculate the Jacobian matrix explicitly and factorize the banded Jacobian from there.

@pkofod
Copy link
Member

pkofod commented Oct 16, 2017

Why would that be superior to simply supplying Jacobians of any type?

@ChrisRackauckas
Copy link
Contributor

It wouldn't.

@pkofod
Copy link
Member

pkofod commented Jan 27, 2018

This should work now. If it doesn't, it's a bug and we'll reopen. The syntax is to supply the OnceDifferentiable instance directly

OnceDifferentiable(f!, j!, fj! x, F, J)

where F is a typical storage for the residuals and J is the banded matrix

@pkofod pkofod closed this as completed Jan 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants