Opened 4 years ago

Closed 4 years ago

#6362 closed enhancement (fixed)

Updating KLU

Reported by: Andreas Heuermann Owned by: Andreas Heuermann
Priority: blocker Milestone: 1.17.0
Component: Run-time Version: v1.17.0-dev
Keywords: klu, sundials Cc:

Description (last modified by Andreas Heuermann)

The klu we use is from SuiteSparse version 3.0 (2007). We recently updated SUNDIALS and should now do the same with klu. The latest version is v5.8.1 from July 2020.

It could be that the old version (that was not tested with this Sundials version) causes problems regarding, see e.g. #6351.

Change History (13)

comment:1 by Andreas Heuermann, 4 years ago

Description: modified (diff)

comment:2 by Francesco Casella, 4 years ago

Priority: normalblocker

comment:4 by Andreas Heuermann, 4 years ago

Resolution: fixed
Status: newclosed

SuiteSparse is now updated.

comment:5 by Francesco Casella, 4 years ago

Weee, nice! Thank you Andreas.

I just started a new Linux build, I'll re-run the tests in #6351 as soon as it is available and report.

comment:6 by Francesco Casella, 4 years ago

Resolution: fixed
Status: closedreopened
Last edited 4 years ago by Francesco Casella (previous) (diff)

comment:7 by Andreas Heuermann, 4 years ago

@sjoelund Changed the Jenkinsfile and restarted the job. Let's see if it passes now:
https://test.openmodelica.org/jenkins/blue/organizations/jenkins/LINUX_BUILDS/detail/LINUX_BUILDS/1201/pipeline

comment:8 by Francesco Casella, 4 years ago

Milestone: 1.18.01.17.0

The nightly was generated successfully, and apparently this also resolved #6351.

Success!

@AnHeuermann, I guess this should be back-ported to 1.17 maintenance, since the new SUNDIALS there could otherwise make trouble. Can you take care of that?

comment:9 by Andreas Heuermann, 4 years ago

Sure, I can backport it. I'm not sure who to do that with 3rd-Party submodule but I'll figure it out.

comment:10 by Andreas Heuermann, 4 years ago

Of course before that I should try to fix #6375.

comment:12 by Andreas Heuermann, 4 years ago

The next build of v1.17 should have the update KLU.

comment:13 by Francesco Casella, 4 years ago

Resolution: fixed
Status: reopenedclosed

Looks good. I'll check some PowerGrids examples with that.

Note: See TracTickets for help on using tickets.