-
Notifications
You must be signed in to change notification settings - Fork 29
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
Incorrect degrees of freedom for Welch's independent t-test [JASP 0.10] #386
Comments
I can verify that the same exact issue exists on the Windows version. It appears to be a new bug, I tested it on 0.9.1.0 and it works fine there. A minor bug, but something to be addressed nevertheless. Keep up the good work guys! |
Thank you both for figuring this out! We'll fix this. |
Thanks for reporting, I fixed it and it will be in the next release. |
I did this (on my own branch) by removing the following line: |
Yes, let's discuss this in person (because what you did implies something
different from what I did)
…On Mon, 17 Jun 2019 at 15:14, AlexanderLyNL ***@***.***> wrote:
I did this (on my own branch) by removing the following line:
https://github.com/jasp-stats/jasp-desktop/blob/e5649a374b7753d56930b82bd007ec0d55757262/JASP-Engine/JASP/R/ttestindependentsamples.R#L351
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#386?email_source=notifications&email_token=ADX2BC372T7X4YYIVHLLHCTP26E2DA5CNFSM4HX3W2UKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX3D4TQ#issuecomment-502677070>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADX2BCZ4GWYYZ7KUMIJ6AY3P26E2DANCNFSM4HX3W2UA>
.
|
First of all, very enthusiastic about the new version! Thanks for the hard work guys!
Without selecting CIs, one obtains the corrected values which should be the case.
The text was updated successfully, but these errors were encountered: