-
Notifications
You must be signed in to change notification settings - Fork 0
ordinal_reg model type with polr engine #4
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Comments
Coordination with the rest of Tidymodels is underway in |
I think that the first thing to figure out is the parameters for
Once we settle on these main arguments and their names, then we can populate |
|
As a new model type and first and probable default engine, my understanding is that these both need to be defined, using separate
make_*()
functions. I've drafted both in this fork branch and raise the issue ahead of submitting a pull request.Should the issue remain open until the entire checklist #3 is completed? If so, then shall we copy the checklist here? (These would be my suggestion.)
(In future, i will raise an issue before doing substantive work, but in this case the work was worth the familiarity-building.)
The text was updated successfully, but these errors were encountered: