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

Effect size and CIs independent of test point #4407

Merged
merged 2 commits into from Dec 2, 2020
Merged

Effect size and CIs independent of test point #4407

merged 2 commits into from Dec 2, 2020

Conversation

AlexanderLyNL
Copy link
Contributor

if (options$VovkSellkeMPR)
result[["VovkSellkeMPR"]] <- .VovkSellkeMPR(p)
if (options[["VovkSellkeMPR"]])
result[["VovkSellkeMPR"]] <- JASP:::.VovkSellkeMPR(p)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The namespace ::: was needed to get things to work in the developer module. I suspect that we can remove JASP::: but I can't check it as I'm unable to build.

AlexanderLyNL added a commit to AlexanderLyNL/jaspTTests that referenced this pull request Nov 24, 2020
- Same as stable: jasp-stats/jasp-desktop#4407
- Removed namespace and dot in front of `VovkSellkeMPR`
@AlexanderLyNL
Copy link
Contributor Author

Approved here:
jasp-stats/jaspTTests#24

@JorisGoosen JorisGoosen merged commit fa02619 into jasp-stats:stable Dec 2, 2020
AlexanderLyNL added a commit to jasp-stats/jaspTTests that referenced this pull request Dec 2, 2020
* Effect size and CIs independent of test point

- Same as stable: jasp-stats/jasp-desktop#4407
- Removed namespace and dot in front of `VovkSellkeMPR`

* Added Don's suggestion on df
@AlexanderLyNL AlexanderLyNL deleted the oneSampleTTestFix branch December 9, 2020 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants