search

cms-analysis / HiggsAnalysis-CombinedLimit

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

Include docker installation index.md #864

Closed nucleosynthesis closed 10 months ago

nucleosynthesis commented 1 year ago

Include docker installation instructions.

This PR likely clashes with #762, which includes installation via VM (do we also want to support that?)

giacomoortona commented 1 year ago

I don't have strong opinions about maintaining the VM option. For sure it is harder to maintain than docker, but there are few user cases where it might be helpful (like students that can access VMs but can't install docker or other softwares). Maybe we can say the default method to have combine standalone is docker, and move other methods (conda, VM,...) to a new section, after a disclaimer saying those are maintained on a best-effort basis and might not be up-to-date?

nucleosynthesis commented 1 year ago

Ok, so now I added the section on combine via CernVM with the instructions pdf into this PR. I couldn't get the merge to work so I just took the edits from #762 directly and added them here. We can close #762.

kcormi commented 1 year ago

@giacomoortona, I have a naive question about the VM setup (as someone who has never used CERN VM before). When I try to visit the page cernvm-online.cern.ch/dashboard, as per the instructions in the pdf, I get a 404 error not found issue, and I cannot find a similar page from the main cernvm.cern.ch page which cernvm-online.cern.ch seems to redirect to.

Am I missing something obvious, or has the setup/website changed?

giacomoortona commented 1 year ago

@kcormi Indeed there were a few changes to CernVM earlier this year, sorry I completely missed that. I need to recover the old config files, redo and test the new instructions. I'll post them as soon as possible. Although, the new instructions are going to be a bit more complicated, so we need to judge if we really want to maintain this option.

kcormi commented 1 year ago

@kcormi Indeed there were a few changes to CernVM earlier this year, sorry I completely missed that. I need to recover the old config files, redo and test the new instructions. I'll post them as soon as possible. Although, the new instructions are going to be a bit more complicated, so we need to judge if we really want to maintain this option.

Okay. I guess the question about maintaining the option depends a little on the use cases we see. I am not sure the target user who would benefit most from the Cern VM option. If we see a target user and a benefit in including it, then perhaps it is worth the effort. But if we don't expect any end users, then perhaps we are better off saving ourselves the work of updating and maintaining the instructions.

giacomoortona commented 1 year ago

I checked the new cernVM workings. They have streamlined a lot of the operations, which are now authomatized. I updated the instructions and removed the (obsolete) slides.