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

Overhaul or remove "About PISA" #800

Open
thehrh opened this issue Aug 12, 2024 · 0 comments
Open

Overhaul or remove "About PISA" #800

thehrh opened this issue Aug 12, 2024 · 0 comments

Comments

@thehrh
Copy link
Contributor

thehrh commented Aug 12, 2024

Apart from the issues that

  • supposedly, "the analysis guide" explains the difference between "parameterized-Monte Carlo (MC) stages and reweighted-MC stages",
  • and that the readme refers to the ezFIT documentation and suggests using it as an alternative analysis framework,

"About PISA" looks more like an attempt to outline the basics of frequentist parameter estimation than a description of PISA. Should it be removed (also given https://github.com/icecube/pisa/blob/master/README.md)?

Keep in mind, however, the general convention (https://github.com/icecube/pisa/blob/master/pisa/general_conventions.md) of "one README.md file within each directory to document the directory's raison d'être, or what is common to all of the things contained in the directory."

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

No branches or pull requests

1 participant