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

Alter the blavaan source #102

Closed
wants to merge 4 commits into from

Conversation

RensDofferhoff
Copy link
Contributor

@RensDofferhoff RensDofferhoff commented Jun 8, 2024

Was there any reason not to use blavaan from cran? @maltelueken

@RensDofferhoff
Copy link
Contributor Author

The unit test numbers are ever so slightly different.
Slight difference in sampling params?
Or was this the reason to use a clone of blavaan?

@maltelueken
Copy link
Collaborator

The reason why we use a clone of blavaan is that after version 0.5-2, a bug was introduced which I could not work around. I saw that they just released a new version 0.5-5, but this seems to introduce new bugs (at least in our module). Previously, it wasn't possible to pin R packages to a specific version, so I used a clone. Is this possible in JASP now?

@maltelueken
Copy link
Collaborator

I am not sure why there are these slight differences in the parameter estimates that make the Bayesian tests fail. They run fine locally. It's likely due to the sampling in Stan indeed. There is a similar issue for the classical tests on Ubuntu, also very small differences.

@RensDofferhoff
Copy link
Contributor Author

Previously, it wasn't possible to pin R packages to a specific version, so I used a clone. Is this possible in JASP now?

I have finished a renv lockfile feature started by @vandenman that allows this.
The only known remaining issue is blavaan compilation on MacOS.
Therefore I had hoped to pin it to precompiled CRAN binaries.

But alas. I will continue to fix the blavaan compilation.

@maltelueken
Copy link
Collaborator

Great! Is it not possible to pin it to 0.5-2 on CRAN then?

@RensDofferhoff
Copy link
Contributor Author

Yes, once/if its merged!

@RensDofferhoff
Copy link
Contributor Author

Though CRAN no longer has those binaries

@maltelueken
Copy link
Collaborator

Ok, but now it still pins to version 0.5-5. I am a bit confused.

@RensDofferhoff
Copy link
Contributor Author

CRAN only keeps the binaries of the latest version.
I will remove this PR for now since we need 0.5-2 due to bugs in later versions.
We can pin it to 0.5-2 once jasp-stats/jasp-desktop#5556 is merged

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.

2 participants