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

Fix description and K==4 case #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xavfunk
Copy link

@xavfunk xavfunk commented Sep 28, 2023

Fixed two things here:

  1. in the function description, line 1 it said that the function would calculate mu's from effect size expressed in Cohen's f, however, the function actually uses eta-squared for this.

  2. in the K==4 case, the vector should be c(-2a, -a, a, 2a), instead of c(-a, -a, a, a). The latter would assume equal levels to have equal means.

Fixed two things here:
1. in the function description, line 1 it said that the function would calculate mu's from effect size expressed in Cohen's f, however, the function actually uses eta-squared for this.

2. in the K==4 case, the vector should be c(-2*a, -a, a, 2*a), instead of c(-a, -a, a, a). The latter would assume equal levels to have equal means.
@arcaldwell49 arcaldwell49 self-assigned this Sep 28, 2023
@arcaldwell49
Copy link
Owner

Thank you for the pull request.

@Lakens, can you verify this? I believe you wrote the original mu_from_ES function?

@Lakens
Copy link
Collaborator

Lakens commented Sep 30, 2023

Hi, this is a helper function that is not used in our paper or app, but that was used in Albers and Lakens, 2018, to set means for a specific simulation. I don't understand the sentence "The latter would assume equal levels to have equal means." (I think it means the -a -a need to be identical, as it was in our simulation, but the proposaed function would make it more generic?). I don't think I made this code either (Casper Albers did) and as far as I know it generates the correct values for the simulation we wanted to run. Maybe @xavfunk can add a bit more explanation of the problem, which error it leads to (or possible which values, which we never used it for, it fails to work for) and how the proposed change in the code fixes the problem that was identified? That would help me to evaluate this. I guess we used this function only very narrowly in situations where it worked, and therefore did not encounter the situation it did not work in. If we can make it work for more feneral calculations, that is useful, perhaps - although the formula really is not meant for general use, as it is limited to k = 4 anyway, just for a very specific simulation we once ran. I also don't remember what paper this was based on, where we got the formula from, or if we created this ourselves.

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