stan_clogit: allow outcome to be factor

[nlpaces]

Summary:

Change acceptable class response variable

Description:

I had assumed that the class of response variable used in the standard clogistic regression function would be accepted in stan_clogit function. clogistic accepts factors. stan_clogit doesn’t. Once I had reformated to numeric, stan_clogit ran without difficulty.

Reproducible Steps:

Message from Jonah:
Good catch! If you want to open an issue at the rstanarm GitHub repo we’ll definitely consider adding support for factors. I think it makes sense to allow them.

RStanARM Version:

2.21.1

R Version:

3.6.3

Operating System:

Ubuntu 18.04.4 LTS

[jgabry]

Thanks for opening the issue. We'll look into allowing the response variable to be a factor.

…

On Mon, May 10, 2021 at 6:16 PM nlpaces ***@***.***> wrote: Summary: Change acceptable class response variable Description: I had assumed that the class of response variable used in the standard clogistic regression function would be accepted in stan_clogit function. clogistic accepts factors. stan_clogit doesn’t. Once I had reformated to numeric, stan_clogit ran without difficulty. Reproducible Steps: Message from Jonah: Good catch! If you want to open an issue at the rstanarm GitHub repo we’ll definitely consider adding support for factors. I think it makes sense to allow them. RStanARM Version: 2.21.1 R Version: 3.6.3 Operating System: Ubuntu 18.04.4 LTS — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#519>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB3PQQ4GGQD4KEMH4WMXWA3TNBZMDANCNFSM44TMBNSQ> .

[jgabry]

This ended up being a really easy fix, so I already have a pull request open: #520. Thanks again for bringing this to our attention.