Skip to content

Fix "std dev" link#204

Merged
corona10 merged 2 commits intomainfrom
stdev_doc
Sep 23, 2024
Merged

Fix "std dev" link#204
corona10 merged 2 commits intomainfrom
stdev_doc

Conversation

@vstinner
Copy link
Copy Markdown
Member

No description provided.

@fzakaria
Copy link
Copy Markdown

Does it help just to emit the standard deviation?

I was reading that knowing the standard deviation for a sample set isn't very helpful; I think it gives off the wrong impression that it's used as a confidence interval.

Reading the link - my feeling was the intent was for it to be a 95% confidence interval especially since the t-test value was calcualte.

@vstinner
Copy link
Copy Markdown
Member Author

The PR fix the documentation of the current implementation: describe correctly what's being used.

@fzakaria
Copy link
Copy Markdown

fzakaria commented Sep 16, 2024

sorry i wasn't clear; the output of stats has a +- using the standard deviation; that gives the impression like it's the confidence interval no?

A title was too short
@vstinner
Copy link
Copy Markdown
Member Author

cc @corona10

@corona10
Copy link
Copy Markdown
Member

I will take a look during the sprint.

Copy link
Copy Markdown
Member

@corona10 corona10 left a comment

Choose a reason for hiding this comment

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

lgtm

@corona10 corona10 merged commit 922ad35 into main Sep 23, 2024
@vstinner vstinner deleted the stdev_doc branch September 23, 2024 18:54
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.

3 participants