Opened 3 years ago

Closed 3 years ago

#6362 closed enhancement (fixed)

Updating KLU

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

Description (last modified by AnHeuermann)

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 Changed 3 years ago by AnHeuermann

  • Description modified (diff)

comment:2 Changed 3 years ago by casella

  • Priority changed from normal to blocker

comment:4 Changed 3 years ago by AnHeuermann

  • Resolution set to fixed
  • Status changed from new to closed

SuiteSparse is now updated.

comment:5 Changed 3 years ago by casella

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 Changed 3 years ago by casella

  • Resolution fixed deleted
  • Status changed from closed to reopened

Unfortunately all the rpm-based nightly builds of Linux are broken ​https://test.openmodelica.org/jenkins/blue/organizations/jenkins/LINUX_BUILDS/detail/LINUX_BUILDS/1198/pipeline/569

Last edited 3 years ago by casella (previous) (diff)

comment:7 Changed 3 years ago by AnHeuermann

@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 Changed 3 years ago by casella

  • Milestone changed from 1.18.0 to 1.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 Changed 3 years ago by AnHeuermann

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 Changed 3 years ago by AnHeuermann

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

comment:12 Changed 3 years ago by AnHeuermann

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

comment:13 Changed 3 years ago by casella

  • Resolution set to fixed
  • Status changed from reopened to closed

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

Note: See TracTickets for help on using tickets.