Skip to content

Expand "A note about sigmas" documentation#292

Merged
dfm merged 5 commits intodfm:mainfrom
jagerber48:sigmas_documentation
Apr 12, 2025
Merged

Expand "A note about sigmas" documentation#292
dfm merged 5 commits intodfm:mainfrom
jagerber48:sigmas_documentation

Conversation

@jagerber48
Copy link
Contributor

The documentation in A note about sigmas is somewhat confusing. See #123. This PR spends more words explaining the difference between a confidence region specified directly by a confidence level and a confidence region specified by the distance from the distribution center in units of sigma (for normally distributed data). Attempts are made to show the pros and cons of two approaches, but to not too strongly endorse one approach over the other.

@@ -77,6 +91,13 @@
"fig = corner.corner(x, quantiles=(0.16, 0.84), levels=(0.68,))\n",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the variables defined above

Copy link
Owner

@dfm dfm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!!

@dfm dfm merged commit 15d79d2 into dfm:main Apr 12, 2025
7 checks passed
@jagerber48 jagerber48 deleted the sigmas_documentation branch April 12, 2025 12:17
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

Successfully merging this pull request may close these issues.

2 participants