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

Version 2.0 Breaking Changes #500

Open
palday opened this issue Sep 29, 2022 · 6 comments
Open

Version 2.0 Breaking Changes #500

palday opened this issue Sep 29, 2022 · 6 comments
Assignees
Milestone

Comments

@palday
Copy link
Member

palday commented Sep 29, 2022

I've noticed that some packages rely on TableRegressionModel to support GLM: jmboehm/RegressionTables.jl#128, yufongpeng/AnovaBase.jl#52 and yufongpeng/AnovaGLM.jl#6. Even if they adapt to support the new approach, we'd better bump version to 2.0 to avoid any breakage. That can also be the occasion to drop some long-deprecated API. We should check whether we would like to make any other breaking changes. (A few other packages use TableRegressionModel for their own models, it would be good that they also stop using it but there's no hurry.)

Originally posted by @nalimilan in #339 (comment)

Here's a quick list of potentially issues that we might want to try to address as part of a push towards 2.0. Several are relatively straightforward, some could potentially be solved via more extensive documentation, and some will require Decisions to be made (e.g. all the stuff with weights).

There are several other issues I would like to see addressed sooner rather than later, but all are technically nonbreaking, at least under ColPrac guidelines, e.g., changes to the show methods, as raised in

@nalimilan nalimilan added this to the 2.0 milestone Sep 29, 2022
@mousum-github
Copy link
Collaborator

mousum-github commented Oct 10, 2022

Right now, we are working on GLM with QR decomposition in two steps

  1. LM with QR
  2. GLM with QR
    and target is to complete by this calendar year.

Hope this will solve some issues related to the PosDefException as mentioned above.

@mousum-github
Copy link
Collaborator

I would like to have Multiple dependent variables, and Quasi Likelihood in GLM 2.0

@nalimilan
Copy link
Member

Nice to hear you're working on QR! I think we can wait until you finish that before tagging 2.0. OTC, multiple dependent variables and quasi-likelihood do not change current behavior so they can be added later (and we have to discuss whether they should live in this package or in a separate one).

@nalimilan
Copy link
Member

I don't think we should do anything about #259. Anyway #487 will change nobs to return an integer, as now the presence of weights is part of the type so there's no type instability. People can use size(modelmatrix(m), 1) to find out the number of rows in the matrix if they need that information.

#483, #255 and #240 would be good to have, but not breaking AFAICT.

@jariji
Copy link

jariji commented Jan 5, 2023

I hope 2.0 fixes #496 and throws an error on missing values to protect users from making analytical errors by accident. lm(...; skipmissing=true) seems fine to me.

@andreasnoack
Copy link
Member

I've split the items to show the status of the issue/PR for each of the changes and also converted the items not already tracked in individual issues to new issues. I've also added them to milestones. Most of them the the 2.0 milestone but some of the issues mentioned in comments as out of scope, I have put on the "out of scope for 2.0" milestone to show that they are deliberately considered out of scope. I hope these steps will make it possible for us to release 2.0 relatively soon. It's overdue.

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

5 participants