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

GroupOrder flag does not seem to work as expected #8

Closed
ManuelaS opened this issue Dec 12, 2019 · 2 comments
Closed

GroupOrder flag does not seem to work as expected #8

ManuelaS opened this issue Dec 12, 2019 · 2 comments

Comments

@ManuelaS
Copy link

Hi,

I tried to reorder the groups from the LAML example to have “N.D.” at one extreme ('Poor', 'Intermediate', 'Good', 'N.D.') using the flag “GroupOrder” and I got the error below.
`
load('laml_RC_data.mat');

[p, fh, stats] = MatSurv(...
laml_RC_TimeVar,...
laml_RC_EventVar,...
laml_RC_GroupVar,...
'GroupOrder', {'Poor', 'Intermediate', 'Good', 'N.D.'})

`

Errors as

Conversion to logical from cell is not possible.
Error in MatSurv (line 293)
if options.GroupOrder

It looks like the flag is expected to get a boolean rather than a cell array with the group order as in the doc string.

@aebergl
Copy link
Owner

aebergl commented Dec 12, 2019

Thanks for finding this! There was some changes pushed that may have broken this and I should have done a more detailed testing. Fixes to come soon

@aebergl
Copy link
Owner

aebergl commented Dec 13, 2019

Fixed. It can now handle both a cell array and a scalar vector as input for GroupOrder.

@aebergl aebergl closed this as completed Dec 13, 2019
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