julia icon indicating copy to clipboard operation
julia copied to clipboard

Reintroduce SparseArrays in the system image

Open ViralBShah opened this issue 3 years ago • 4 comments

Reverting https://github.com/JuliaLang/julia/pull/44247 in order to avoid the impact on startup time for several packages. The impact is about 2 seconds, but it affects several packages that do not always need SparseArrays, but are forced to have that dependency due to lack of conditional dependencies, which we hope to have in 1.10.

We do not need to reintroduce SuiteSparse separately since it was reintegrated into SparseArrays.jl in order to maintain backwards compatibility.

cc @Wimmerer

ViralBShah avatar Aug 07 '22 23:08 ViralBShah

I don't think this needs backport. It's in the sysimage in 1.8

IanButterworth avatar Aug 07 '22 23:08 IanButterworth

I expected this. Is this the plan for all stdlibs that were being removed?

rayegun avatar Aug 07 '22 23:08 rayegun

We still hope to keep DelimitedFiles in 1.9 out of the sysimage, and hopefully also remove it all the way from stdlib. That should give us some experience when it comes to SparseArrays for 1.10. @KristofferC may be able to say more.

ViralBShah avatar Aug 07 '22 23:08 ViralBShah

I can't quite tell what is causing the failing darwin x64 test. Doesn't feel related to this PR.

ViralBShah avatar Aug 08 '22 12:08 ViralBShah

@SobhanMP Heads up.

ViralBShah avatar Aug 10 '22 13:08 ViralBShah

We still hope to keep DelimitedFiles in 1.9 out of the sysimage, and hopefully also remove it all the way from stdlib. That should give us some experience when it comes to SparseArrays for 1.10. @KristofferC may be able to say more.

Why does it need to be removed?

A-childs-encyclopedia avatar Apr 06 '23 07:04 A-childs-encyclopedia

So we can bump the package w/o bumping Julia or needing to backport (or at least make the process easier).

SobhanMP avatar Apr 07 '23 05:04 SobhanMP