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

docs: Add citations from Ph.D. theses on CDS #2120

Merged
merged 4 commits into from
Feb 17, 2023

Conversation

matthewfeickert
Copy link
Member

@matthewfeickert matthewfeickert commented Feb 17, 2023

Description

Add citations from Ph.D. theses that are on CDS but not yet on INSPIRE:

(belated) Congratulations Dr. Stanislaus and Dr. Hesping! 🚀

Checklist Before Requesting Reviewer

  • Tests are passing
  • "WIP" removed from the title of the pull request
  • Selected an Assignee for the PR to be responsible for the log summary

Before Merging

For the PR Assignees:

  • Summarize commit messages into a comprehensive review of the PR
* Add general citation from 'Searching for Beyond the Standard Model Resonances
  in the HH→bb¯bb¯ Final State Using the ATLAS Detector'.
   - c.f. https://cds.cern.ch/record/2743178/
   - Ph.D. dissertation of Beojan Stanislaus
* Add use citation from 'Differential Cross Section Measurement of the
  pp→WH→WWW Process With the ATLAS Experiment'.
   - c.f. https://cds.cern.ch/record/2799614/
   - Ph.D. dissertation of Moritz Hesping

* Add use citation from 'Searching for Beyond the Standard Model Resonances
  in the HH→bb¯bb¯ Final State Using the ATLAS Detector'.
   - c.f. https://cds.cern.ch/record/2743178/
   - Ph.D. dissertation of Beojan Stanislaus
* Add use citation from 'Differential Cross Section Measurement of the
  pp→WH→WWW Process With the ATLAS Experiment'.
   - c.f. https://cds.cern.ch/record/2799614/
   - Ph.D. dissertation of Moritz Hesping
@matthewfeickert matthewfeickert added the docs Documentation related label Feb 17, 2023
@matthewfeickert matthewfeickert self-assigned this Feb 17, 2023
@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Base: 98.30% // Head: 96.49% // Decreases project coverage by -1.81% ⚠️

Coverage data is based on head (1d9db5e) compared to base (74441ab).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2120      +/-   ##
==========================================
- Coverage   98.30%   96.49%   -1.81%     
==========================================
  Files          69       69              
  Lines        4531     4531              
  Branches      645      608      -37     
==========================================
- Hits         4454     4372      -82     
- Misses         45      123      +78     
- Partials       32       36       +4     
Flag Coverage Δ
contrib ?
doctest ?
unittests-3.10 ?
unittests-3.8 96.33% <ø> (ø)
unittests-3.9 96.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/pyhf/contrib/viz/brazil.py 15.38% <0.00%> (-84.62%) ⬇️
src/pyhf/contrib/utils.py 65.90% <0.00%> (-34.10%) ⬇️
src/pyhf/optimize/__init__.py 95.45% <0.00%> (-4.55%) ⬇️
src/pyhf/tensor/pytorch_backend.py 94.69% <0.00%> (-3.79%) ⬇️
src/pyhf/infer/test_statistics.py 96.66% <0.00%> (-3.34%) ⬇️
src/pyhf/tensor/tensorflow_backend.py 95.33% <0.00%> (-2.00%) ⬇️
src/pyhf/tensor/jax_backend.py 97.91% <0.00%> (-0.70%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@beojan
Copy link
Contributor

beojan commented Feb 17, 2023

The citation in my thesis is for the pyhf documentation (thanks for the really clear description of the HistFactory likelihood function by the way!) rather than the software.

The resonant version of that analysis (which is what's described in my thesis) still used HistFactory. My understanding is that the non-resonant version does use pyhf, but I can't say for sure since I'm not involved in that one.

@matthewfeickert matthewfeickert changed the title docs: Add use citations from Ph.D. theses on CDS docs: Add citations from Ph.D. theses on CDS Feb 17, 2023
@matthewfeickert
Copy link
Member Author

The citation in my thesis is for the pyhf documentation (thanks for the really clear description of the HistFactory likelihood function by the way!) rather than the software.

Thanks for the info there! Giordon had noticed the citation and mentioned it to me and I hadn't bothered to properly check myself what kind of citation when writing this up.

@matthewfeickert
Copy link
Member Author

@matthewfeickert
Copy link
Member Author

I'm going to approve and merge this myself. As always, PRs approved by a single core dev can be reverted as needed by the rest of the dev team.

@matthewfeickert matthewfeickert merged commit 4377bdf into main Feb 17, 2023
@matthewfeickert matthewfeickert deleted the docs/add-cds-thesis-citations branch February 17, 2023 22:53
matthewfeickert added a commit that referenced this pull request Apr 6, 2023
* Backport:
   - PR #2040
   - PR #2063
   - PR #2066
   - PR #2082
   - PR #2083
   - PR #2089
   - PR #2092
   - PR #2094
   - PR #2101
   - PR #2107
   - PR #2116
   - PR #2120
   - PR #2126
   - PR #2127
   - PR #2153
   - PR #2160
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation related
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants