cms-analysis / HiggsAnalysis-CombinedLimit

CMS Higgs Combination toolkit.
https://cms-analysis.github.io/HiggsAnalysis-CombinedLimit/latest
Apache License 2.0
75 stars 380 forks source link

Standalone installation description not working on lxplus #923

Closed pfackeldey closed 5 months ago

pfackeldey commented 5 months ago

Dear Combine Team,

the standalone compilation instructions yield a segfault on lxplus (see: https://cms-analysis.github.io/HiggsAnalysis-CombinedLimit/#standalone-compilation). Likely due to the fact that the standalone script is still relying on the slc7 arch (@anigamova).

Best, Peter

nucleosynthesis commented 5 months ago

If you have access, you can compile inside the SLC7 environment using apptainer and then source the standalone script outside to use combine (see https://cms-sw.github.io/singularity.html) i.e

cmssw-el7
. env_standalone.sh
make -j 4
exit
source . env_standalone.sh
combine --help

should work on lxplus

pfackeldey commented 5 months ago

Yes, the line cmssw-el7 is just missing in the docs for the standalone installation.

nucleosynthesis commented 5 months ago

Ok, in that case @anigamova / @kcormi , can we have the line added at least until we update to v10 when we shouldn't need it anymore?

kcormi commented 5 months ago

Here is a PR #924 addressing this. I prefer not to change the primary standalone compilation instructions as not everyone uses lxplus.

kcormi commented 5 months ago

924 has been merged, so those instructions are now on the page! thanks for the feedback.