Skip to content

GLM model selection #83

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

Open
OriolAbril opened this issue Mar 30, 2021 · 9 comments
Open

GLM model selection #83

OriolAbril opened this issue Mar 30, 2021 · 9 comments
Labels
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-model-selection.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
Copy link
Member Author

I tried moving from nbsphinx to myst-nb and was unable to do so because the widgets in this notebook are not saved correctly, see https://github.com/jupyter/jupyter-sphinx/blob/master/jupyter_sphinx/ast.py#L607. After manually editing the metadata to remove the widgets completely I was able to build the documentation.

Side note: the DE-MCMC notebooks also have widgets but they are correctly saved are are not a problem.

@chiral-carbon
Copy link
Collaborator

@OriolAbril can I work on this?

@chiral-carbon chiral-carbon self-assigned this Jun 8, 2021
@OriolAbril
Copy link
Member Author

Great!

@drbenvincent
Copy link
Contributor

This could do with some simplification to focus on the core aspects of model comparison. Specific proposals:

  • remove interactive aspects of this notebook. They only work when running locally, not when viewing on the website. And in fact it is pretty clunky when I run the ipywidgets locally. Seems like a distraction.
    • remove interactive the dataset viewing
    • remove the interactive posterior plot viewing and just show static posterior predictions
  • remove plot_annotated_trace and just use az.plot_trace
  • why are we sometimes using raw data, sometimes the standardised data?

@OriolAbril
Copy link
Member Author

Always in favour of simplifying notebooks to have a clearer and more to the point scope. My main concern (not remembering the content of either notebook right now so tread lightly) is that it might end up being too similar to https://www.pymc.io/projects/docs/en/stable/learn/core_notebooks/model_comparison.html

@drbenvincent
Copy link
Contributor

Good point. Could potentially be expanded to include model comparison by Savage Dickey method perhaps

@drbenvincent
Copy link
Contributor

I didn't actually realise there were these separate core notebooks.

I think it's very cool to have these core notebooks under "Learn". But I think these notebooks should additionally be visible along with the regular pymc example notebooks. Some people might go straight to the Examples and not know about the extra ones under Learn.

Can I propose that these core notebooks stay where they are, but are additionally available with the rest of the Example notebooks?

@OriolAbril
Copy link
Member Author

But I think these notebooks should additionally be visible along with the regular pymc example notebooks

Yes, they should be listed at the top of the gallery page. We can use cards manually with sphinx-design

@drbenvincent
Copy link
Contributor

I'll submit an issue and PR for this. Hopefully in the next few days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tracker id Issues used as trackers in the notebook update project, do not close!
Projects
Status: Best practices (v3)
Development

No branches or pull requests

3 participants