Skip to content

Options#103

Merged
ecmerkle merged 2 commits intoecmerkle:masterfrom
emstruong:options
Jan 4, 2026
Merged

Options#103
ecmerkle merged 2 commits intoecmerkle:masterfrom
emstruong:options

Conversation

@emstruong
Copy link
Contributor

Concerns #102

Seems to be fine AFAICT, but I'm not sure if the tests ran properly....

I don't know if this is a temporary issue, but apparently bsem isn't even running on my system right now for whatever odd reason. But I can't imagine how my changes would've affected that.

Alarmingly, even if I checkout a much earlier commit far before these particular changes, I still can't get bsem working right now.

I have to switch to something else right now, but perhaps this initial proposal might be helpful.

@ecmerkle ecmerkle merged commit ff7cd8f into ecmerkle:master Jan 4, 2026
3 of 4 checks passed
@ecmerkle
Copy link
Owner

ecmerkle commented Jan 4, 2026

Thanks!

The tests are working locally for me at the moment, though I notice that something is irritating the github action. I will have to look at it some more. If you have the cmdstan option set on your machine, I could imagine that some test might fail. This is because the tests currently focus heavily on the default target = "stan" (target = "cmdstan" has been undocumented up to now). For this reason, I will comment out the suggestion about options(blavaan.target = "cmdstan") until I can deal with the tests.

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