Skip to content

GLM logistic #82

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

Closed
OriolAbril opened this issue Mar 30, 2021 · 10 comments
Closed

GLM logistic #82

OriolAbril opened this issue Mar 30, 2021 · 10 comments
Assignees
Labels
high impact Notebooks with most visits on docs.pymc.io tracker id Issues used as trackers in the notebook update project, do not close!

Comments

@OriolAbril
Copy link
Member

File: https://github.com/pymc-devs/pymc-examples/blob/main/examples/generalized_linear_models/GLM-logistic.ipynb
Reviewers:

The sections below may still be pending. If so, the issue is still available, it simply doesn't
have specific guidance yet. Please refer to this overview of updates

Known changes needed

Changes listed in this section should all be done at some point in order to get this
notebook to a "Best Practices" state. However, these are probably not enough!
Make sure to thoroughly review the notebook and search for other updates.

General updates

ArviZ related

Changes for discussion

Changes listed in this section are up for discussion, these are ideas on how to improve
the notebook but may not have a clear implementation, or fix some know issue only partially.

General updates

ArviZ related

Notes

Exotic dependencies

Computing requirements

@OriolAbril OriolAbril added the tracker id Issues used as trackers in the notebook update project, do not close! label Mar 30, 2021
@OriolAbril OriolAbril added the high impact Notebooks with most visits on docs.pymc.io label Apr 7, 2021
@twiecki
Copy link
Member

twiecki commented Jul 22, 2021

This is also using the deprecated glm submodule which we should remove and implement a logistic regression directly.

@chiral-carbon
Copy link
Collaborator

hey! can I take this up?

@twiecki
Copy link
Member

twiecki commented Jul 22, 2021 via email

@drbenvincent
Copy link
Contributor

I can update this to v4

@drbenvincent drbenvincent self-assigned this May 31, 2022
@drbenvincent
Copy link
Contributor

@OriolAbril / @twiecki
I've started to port this, but noticed that it overlaps a lot with an official Bambi example (see https://bambinos.github.io/bambi/main/notebooks/model_comparison.html)

I will definitely link to that, but I'm wondering if this notebook is essentially redundant?

Let me know if we should kill this or just continue to port what is there.

@ghost
Copy link

ghost commented May 31, 2022

I vote in favor of removing. We should take advantage of any opportunity to reduce the size of the example library.

@aloctavodia
Copy link
Member

Agreed. Is there a place in PyMC were we redirect/point to bambi for GLMs?

@drbenvincent
Copy link
Contributor

drbenvincent commented May 31, 2022

Agreed. Is there a place in PyMC were we redirect/point to bambi for GLMs?

Some of the GLM notebooks do hand coded PyMC models. Some include Bambi only. Some contain a mix.

If we were going to move towards being consistent, my preference would be to focus on hand coded PyMC models but with linking to Bambi and possibly including non-executed Bambi code in markdown cells.

Why? I think it makes sense to avoid creating a dependency upon the Bambi API. If it changes then we have to update the PyMC examples. This also then creates redundancy. It also goes against the whole 'lego block' approach of building your own models with PyMC.

I think Bambi is fantastic, but for the PyMC examples, my feeling is that we should avoid Bambi only notebooks.

But in the mean time I'll open a PR to remove this notebook.

@twiecki
Copy link
Member

twiecki commented May 31, 2022

Closed via #345.

@twiecki twiecki closed this as completed May 31, 2022
@aloctavodia
Copy link
Member

Totally agree about focusing on hand coded model here even more we could have the handcoded version here and the formula version in bambi doc and mutually crossrefrence notebooks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high impact Notebooks with most visits on docs.pymc.io tracker id Issues used as trackers in the notebook update project, do not close!
Projects
Development

No branches or pull requests

5 participants