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

Change default extraction method for factor analysis to be based on factors (instead of principal components) #199

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

patc3
Copy link
Contributor

@patc3 patc3 commented Dec 19, 2023

This also clarifies the labels for the parallel analysis, which were adjusted recently for the principal components analysis but not the factor analysis.

@juliuspfadt
Copy link
Collaborator

Thank you @patc3 for the PR. I agree that it is inconsistent to have EFA and then run the parallel analysis by default based on PCs, however I think we had a reason for this, which is that it performs better, and I think it was based on this paper: http://dx.doi.org/10.1037/met0000200. we had a somewhat related discussion about these types of inconsistencies here jasp-stats/jasp-issues#2418. In general, I am planning to split the analyses into one to determine the number of factors/components and one the runs EFA or PCA. I think we would then not run into these types of inconsistencies. But I dont know when I will get around to it.
Anyways, I am not feeling very strongly about the default parallel analysis method.

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.

None yet

2 participants