ComPWA / polarimetry

Polarimetry for the decay Λc → p K π
https://compwa.github.io/polarimetry
Apache License 2.0
5 stars 1 forks source link

MAINT: outsource to Lc2ppiKSemileptonicModelLHCb.jl #326

Closed redeboer closed 7 months ago

redeboer commented 7 months ago

Closes #325

redeboer commented 7 months ago

161e5b5 updates Manifest.toml. Running the Pluto notebooks works without errors, but I did get the impression that some cells crash.

redeboer commented 7 months ago

Next step is to recreate the environment under the julia/ directory with only Pluto.jl and Lc2ppiKSemileptonicModelLHCb.jl. I tried to do this as follows:

rm julia/*.toml
julia --project=julia

Then start the package manager and ran

(julia) pkg> add https://github.com/mmikhasenko/Lc2ppiKSemileptonicModelLHCb.jl

but got

    Updating git-repo `https://github.com/mmikhasenko/Lc2ppiKSemileptonicModelLHCb.jl`
   Resolving package versions...
ERROR: Unsatisfiable requirements detected for package ThreeBodyDecay [e6563dab]:
 ThreeBodyDecay [e6563dab] log:
 ├─ThreeBodyDecay [e6563dab] has no known versions!
 └─restricted to versions 0.6 by Lc2ppiKSemileptonicModelLHCb [2be14d12] — no versions left
   └─Lc2ppiKSemileptonicModelLHCb [2be14d12] log:
     ├─possible versions are: 0.1.0 or uninstalled
     └─Lc2ppiKSemileptonicModelLHCb [2be14d12] is fixed to version 0.1.0
mmikhasenko commented 7 months ago

seems to be julia issue with discovering unregistered packages, should be solved by installing TBD separately

]
add https://github.com/mmikhasenko/ThreeBodyDecay.jl
add https://github.com/mmikhasenko/Lc2ppiKSemileptonicModelLHCb.jl
# works afterwords
using Lc2ppiKSemileptonicModelLHCb
model = published_model("Default amplitude model")
redeboer commented 7 months ago

@mmikhasenko please have a look at this log https://github.com/ComPWA/polarimetry/actions/runs/7223081546/job/19681403799?pr=326 I don't get Julia's package dependency management. Shouldn't that dependency be pulled in from Lc2ppiKSemileptonicModelLHCb.jl?

redeboer commented 7 months ago

Nvm the scripts just need to be updated 😅

mmikhasenko commented 7 months ago

thanks for the julia test! Works locally for me, let's see if CI works

redeboer commented 7 months ago

Cool thanks, seems to work now