Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates for the x-section page #74

Open
katilp opened this issue Apr 24, 2022 · 3 comments
Open

Updates for the x-section page #74

katilp opened this issue Apr 24, 2022 · 3 comments
Assignees

Comments

@katilp
Copy link
Member

katilp commented Apr 24, 2022

Thanks @jieunyoo for putting together the x-section page! I have some minor comments and suggestions:

  • doing cmsenv in the current containers should be avoided as the environment is already set at the startup with some updates with respect to what cmsenv does. E.g. git clone https:... and curl should work in the containers if cmsenv is not done

  • it is not necessary to copy the data file to the container, the files should be reachable directly with

    cmsRun ana.py inputFiles="root://eospublic.cern.ch//eos/opendata/cms/....

  • for the ease of reading and for the first try, we could also have full command with the example file name (with the full file path which one can get in the correct format from the record file listing)

  • maybe a screenshot of the listing (or it can also be copied verbatim) would be useful to have

Maybe @caredg has some further suggestions?

@katilp
Copy link
Member Author

katilp commented Apr 24, 2022

Maybe also add a mention of the first Caveat in https://twiki.cern.ch/twiki/bin/viewauth/CMS/HowToGenXSecAnalyzer#Caveats i.e.:

The cross-sections found with this tool are those predicted by the respective generators. There may be better estimates, coming from dedicated task forces, theory papers etc.

@caredg
Copy link
Collaborator

caredg commented Apr 26, 2022

Thanks @jieunyoo! The only thing I would change is to put this description under Monte Carlo Simulations. I would put it just as a subsection and not necessarily as a page itself (but I leave it to your best judgement). The one thing I would not do is to leave it under the Interpretation section. I suppose that section has a completely different idea and it has to do more with actual measurements of the cross sections (and not xsec extractions for already simulated samples.) Thanks!

jieunyoo added a commit to jieunyoo/cms-opendata-guide that referenced this issue Apr 27, 2022
jieunyoo added a commit to jieunyoo/cms-opendata-guide that referenced this issue Apr 27, 2022
jieunyoo added a commit that referenced this issue Apr 27, 2022
edits to address issue #74. syntax changed to directly access file instead of copying file to container, screenshot included
@jieunyoo
Copy link
Collaborator

Hi. I updated the page, but the screenshot is a bit small due to the column formatting. If you would like it removed or other changes, please let me know .Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants