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.

Sign up for GitHub

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

Jump to bottom

Remove sumabs2j #67

Merged

sglyon merged 2 commits into JuliaNLSolvers:master from matthieugomez:virew

Sep 5, 2016

Contributor

matthieugomez commented Sep 4, 2016 •

edited

sumabs2j is not needed anymore because view of sparse matrices is performant in 0.5


          remove colsumabs2j

2f9ef8c

Collaborator

KristofferC commented Sep 4, 2016

Are they performant in 0.4 with Compat?

Contributor Author

matthieugomez commented Sep 4, 2016

yes. It calls slice that works well too. I'm not sure when it changed.


          update compat

572f245

sglyon merged commit e41f170 into JuliaNLSolvers:master

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