Skip to content

Conversation

@JWIgnatius
Copy link

PR Summary

Measure-Object -StandardDeviation is currently doing the sample standard deviation meaning division by n-1 instead of n so update it to do the standard deviation.

PR Context

I was using the StandardDeviation measure as a calculator to confirm some calculations online and found inconsistencies which were due to the sample standard deviation being used. When calculating the standard deviation I don't think people are expecting the sample standard deviation.

PR Checklist

joe walsh added 2 commits October 24, 2024 00:08
@JWIgnatius
Copy link
Author

@microsoft-github-policy-service agree

@JWIgnatius JWIgnatius marked this pull request as ready for review October 24, 2024 00:05
@iSazonov
Copy link
Collaborator

It is a breaking change. This should be discussed in advance. Please open new issue.
Since the link to the algorithm is explicitly specified in the code, then without a doubt it is by-design. And the easiest thing is to fix the documentation.

@JWIgnatius
Copy link
Author

Thanks @iSazonov, I've created an issue for it #24487

@microsoft-github-policy-service microsoft-github-policy-service bot added the Review - Needed The PR is being reviewed label Oct 31, 2024
@microsoft-github-policy-service
Copy link
Contributor

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

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

Labels

Review - Needed The PR is being reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants