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

EFA: rename 'target' rotation 'target.strict' rotation #384

Open
yrosseel opened this issue Sep 22, 2024 · 1 comment
Open

EFA: rename 'target' rotation 'target.strict' rotation #384

yrosseel opened this issue Sep 22, 2024 · 1 comment

Comments

@yrosseel
Copy link
Owner

When performing 'target' rotation, you need to provide a target matrix. This matrix often consists of ones and zeroes, and the applied user often regards this matrix as a 'pattern' matrix, indicating zero and non-zero values. But the current 'target' rotation algorithm takes those ones literally, often resulting in poor results. The 'trick' is to replace all the ones by 'NA'.
Perhaps we should rename 'target' to 'target.strict', and when 'target' rotation is requested, we will replace all '1' values by 'NA' and effectively perform partial target rotation instead, as this is, typically, what users expect.

@FlorianScharf
Copy link

I like this suggestion very much, it aligns better with the descriptions of this method in papers. Also I think it is safe to assume that no one actually aims for target loadings of 1.
I have another observation: I think even if users actually want to do a target rotation, they tend to think of their target in standardized loadings which is not how this method works. I think we should more clearly instruct users that standardized targets only work when the data input is also standardized.

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

No branches or pull requests

2 participants