-
Notifications
You must be signed in to change notification settings - Fork 11
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 error messages #59
Fix error messages #59
Conversation
e4434cf
to
27a624a
Compare
98f9bdc
to
add2656
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code changes look ok to me, but there are some general issues:
There still some kind of state issue; In this file (change extension from .jasp
to .zip
): process-bug.zip
I get Error in object@Options: no applicable method for '@' applied to an object of class 'NULL'
, right when I created the second model
If I refresh the analysis, I get a different error. Then if I remove the first model, the second model runs fine.
Otherwise, the tests fail quite catastrophically on Ubuntu. I assume you did not have the opportunity to test it there?
I think we could merge this, but tt would be good to make sure that the module is not broken for Linux users for the release...
88b87ea
to
71bc1ea
Compare
Thanks for the review! I fixed the state issue (it was actually a container issue) and the issue with the summary table if one model is not estimated yet (invalid). I also used the Turns out that containers for new models were not correctly added to I also found out that the tests for models 82-92 were not correct (missing input), which I fixed. |
Regardint the failing tests on Linux: This is happening since JASP moved to R 4.3.1, so I think it might be a seed issue. I will try to run the tests on my WSL to see if they also fail there. EDIT: There are two issues I think:
Both issues are due to lavaan I think, as I could reproduce the differences in lavaan without the module. |
I fixed a bug that caused the large differences between Linux and the other systems for some failed tests. The bug was that covariances where estimated between mediator variables that also had a causal path between them. |
Fixes the error message for Hayes models that are not implemented to appear correctly (previously did not appear at all).
Fixes the error message when a linear local test type is chosen but a factor variable is in the model.
Removes the footnote about model saturation from the model summary table.
Changes the footnote about model converge to appear as a symbol for the corresponding models in the model summary table.
Adds a disclaimer about causal effects to parameter estimates output.