Compat error with LinearOperatorCOllection v1.2.0

[aTrotier]

The API to LinearOperatorCollection.il had changed in the most recent version (V1.2.0).
Therefore, in reconstruction multiCoilMultiEcho subspace (IterativeReconstruction.il)
the call to Weighting Op on L374 returns a "MethodError: no method matching" error. This is because a ";" is missing from the WeightingOp function definition in WeightingOp.jl (LinearOperatorCollection.jl), and so the keyword arguments are unsupported in LinearOperatorCollection.jl V1.2.0
By Downgrading to V1.1.2, I was able to run the code sucessfully

The author's own code appeared to run entirely as expected and without issues.

linked to JuliaImageRecon/LinearOperatorCollection.jl#7

[aTrotier]

The API was changed between my last test and the review.

A compat entry to LinearOperatorCollection will be added and the manifest to generate the figure will be stored in the docs folder for reproducibility purpose.

[aTrotier]

@wtclarke I guess that you reviewed my package. If it is the case, thanks for the time you take to make it works and for the the comments 👍

If you have time, can you try to reproduce the figure with the release 1.0.1.
I have pin the version to "=1.1.2" in the package and I have also put my Manifest.toml in the docs folder.

If you launch julia in the docs folder julia --project -t auto and run

using Pkg
Pkg.instantiate()

It should always use the same version of the packages.

PS : for the CairoMakie missing package, I think you were not using the Project.toml in docs, it was not clear in the readme. I have added a sentence about that.

[wtclarke]

You guessed correctly. ;)

It seems to work now, though I'm not entirely sure I'm working with a clean environment now. Is there a way to ensure that?

Yes, I hadn't gathered I had to run from within the docs folder, though I should have guessed from the path.

[aTrotier]

I think you can delete the Manifest.toml.