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

FAQ page first draft #657

Merged
merged 8 commits into from
Apr 18, 2023
Merged

FAQ page first draft #657

merged 8 commits into from
Apr 18, 2023

Conversation

abuzarmahmood
Copy link
Contributor

Adding questions for FAQ page per Issue #644

@abuzarmahmood abuzarmahmood marked this pull request as draft March 23, 2023 16:28
Copy link
Collaborator

@canyon289 canyon289 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is look good to me. I'll let Tomas review as well!

Thank you for the PR

docs/faq.rst Outdated Show resolved Hide resolved
docs/faq.rst Outdated Show resolved Hide resolved
docs/faq.rst Show resolved Hide resolved
docs/faq.rst Outdated Show resolved Hide resolved
docs/faq.rst Outdated

What is the difference between Bambi and PyMC3?
----------------------------
* Bambi is a wrapper around PyMC3, NumPyro, and BlackJax. It provides a simple interface for
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would say Bambi is a library built on top of PyMC, it's not a wrapper of PyMC. Also, it's not a wrapper of NumPyro or BlackJax. Technically when PyMC compiles a PyMC model for sampling with NumPyro or Blackjax it's creating such models, but it's happening all automatically and it's not exposed to users

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, calling it a wrapper is incorrect. Thank you for the correction. I also took out mention of numpyro and blackjax here because they are mentioned as backends in multiple other questions.

@tomicapretto
Copy link
Collaborator

tomicapretto commented Mar 23, 2023

Hi @abuzarmahmood thanks a lot for this contribution!

I think the following questions fit really well in the FAQ

  • What is the difference between Bambi and PyMC3?
  • What sampling methods are available?
  • Can infernece in Bambi be sped up using GPUs/TPUs?
  • My data has a non-normal distributions, can I still use Bambi?
    • Although I would phrase this in a different way.
    • I want to use a non-normal likelihood model, can I still use Bambi?
  • How do I find out what priors are available?
  • Does bambi come with pre-specified regression models?

I'm not so sure about these other questions

  • Why have a Bayesian regression library?
  • I'm not familiar with Bayesian modelling, where can I learn more?
  • My sampler through errors/indicating divergences, what should I do?

Those are more conceptual/philosophical and modeling-related questions. I don't think we should try to answer them in the FAQ, since I think the FAQ is for questions about the library.

Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>
@codecov-commenter
Copy link

codecov-commenter commented Mar 23, 2023

Codecov Report

Merging #657 (2733fbd) into main (afcabb6) will increase coverage by 0.06%.
The diff coverage is 95.45%.

❗ Current head 2733fbd differs from pull request most recent head 4d829ab. Consider uploading reports for the commit 4d829ab to get more accurate results

@@            Coverage Diff             @@
##             main     #657      +/-   ##
==========================================
+ Coverage   87.56%   87.62%   +0.06%     
==========================================
  Files          40       40              
  Lines        2629     2650      +21     
==========================================
+ Hits         2302     2322      +20     
- Misses        327      328       +1     
Impacted Files Coverage Δ
bambi/defaults/distributions.py 100.00% <ø> (ø)
bambi/defaults/families.py 87.50% <ø> (ø)
bambi/defaults/utils.py 87.50% <ø> (ø)
bambi/families/likelihood.py 85.36% <ø> (ø)
bambi/families/univariate.py 90.37% <95.00%> (+0.80%) ⬆️
bambi/families/family.py 93.25% <100.00%> (ø)

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@abuzarmahmood
Copy link
Contributor Author

Hi @abuzarmahmood thanks a lot for this contribution!

I think the following questions fit really well in the FAQ

  • What is the difference between Bambi and PyMC3?

  • What sampling methods are available?

  • Can infernece in Bambi be sped up using GPUs/TPUs?

  • My data has a non-normal distributions, can I still use Bambi?

    • Although I would phrase this in a different way.
    • I want to use a non-normal likelihood model, can I still use Bambi?
  • How do I find out what priors are available?

  • Does bambi come with pre-specified regression models?

I'm not so sure about these other questions

  • Why have a Bayesian regression library?
  • I'm not familiar with Bayesian modelling, where can I learn more?
  • My sampler through errors/indicating divergences, what should I do?

Those are more conceptual/philosophical and modeling-related questions. I don't think we should try to answer them in the FAQ, since I think the FAQ is for questions about the library.

That makes sense. I've updated it to include the questions you suggested.

@abuzarmahmood abuzarmahmood marked this pull request as ready for review March 23, 2023 22:37
docs/faq.rst Outdated Show resolved Hide resolved
Copy link
Collaborator

@tomicapretto tomicapretto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @abuzarmahmood sorry for the delay in the reply. I just requested one more small change, then we're good to merge. Thanks!

@abuzarmahmood
Copy link
Contributor Author

abuzarmahmood commented Apr 18, 2023

@tomicapretto Got it! That was my bad for confusing families with priors. I've update the response to the priors question (67f52fb).

@tomicapretto
Copy link
Collaborator

@tomicapretto Got it! That was my bad for confusing families with priors. I've update the response to the priors question (67f52fb).

Last nit: It's PyMC instead of PyMC3 (we're using PyMC version 5, but we write just PyMC)

@abuzarmahmood
Copy link
Contributor Author

@tomicapretto Got it! That was my bad for confusing families with priors. I've update the response to the priors question (67f52fb).

Last nit: It's PyMC instead of PyMC3 (we're using PyMC version 5, but we write just PyMC)

Sounds good. Latest commit uses "PyMC" instead of "PyMC3".

@tomicapretto
Copy link
Collaborator

The test failure is known. It's due to an update of the variable names in PyMC/PyTensor, which uses full names now. It's already updated in #668. I'm merging.

Thanks @abuzarmahmood !

@tomicapretto tomicapretto merged commit 6f2f399 into bambinos:main Apr 18, 2023
@abuzarmahmood
Copy link
Contributor Author

Lovely. Thank you @tomicapretto @canyon289 for your guidance

GStechschulte pushed a commit to GStechschulte/bambi that referenced this pull request Apr 24, 2023
* FAQ page first draft

* Update docs/faq.rst

Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>

* Update docs/faq.rst

Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>

* Remove general bayesian modelling questions + minor fixes

* Update faq.rst

* Update faq.rst

---------

Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>
tomicapretto added a commit that referenced this pull request Apr 27, 2023
* plot_cap using post. pred. samples

* add mtcars dataset and test plot_cap using post. pred. samples

* plot_cap show predictions at obs. level

* remove unused code and formatting

* Add mtcars as a dataset. Bump PyMC to 5.3.0

* FAQ page first draft (#657)

* FAQ page first draft

* Update docs/faq.rst

Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>

* Update docs/faq.rst

Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>

* Remove general bayesian modelling questions + minor fixes

* Update faq.rst

* Update faq.rst

---------

Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>

* black reformatting

* add tests for pps=bool using pytest parameterization

---------

Co-authored-by: Tomas Capretto <tomicapretto@gmail.com>
Co-authored-by: Abuzar Mahmood <abuzarmahmood@gmail.com>
Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>
GStechschulte added a commit to GStechschulte/bambi that referenced this pull request May 9, 2023
…inos#668)

* plot_cap using post. pred. samples

* add mtcars dataset and test plot_cap using post. pred. samples

* plot_cap show predictions at obs. level

* remove unused code and formatting

* Add mtcars as a dataset. Bump PyMC to 5.3.0

* FAQ page first draft (bambinos#657)

* FAQ page first draft

* Update docs/faq.rst

Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>

* Update docs/faq.rst

Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>

* Remove general bayesian modelling questions + minor fixes

* Update faq.rst

* Update faq.rst

---------

Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>

* black reformatting

* add tests for pps=bool using pytest parameterization

---------

Co-authored-by: Tomas Capretto <tomicapretto@gmail.com>
Co-authored-by: Abuzar Mahmood <abuzarmahmood@gmail.com>
Co-authored-by: Ravin Kumar <7213793+canyon289@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

4 participants