Skip to content

Exploring VI implementations #7799

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

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

jessegrabowski
Copy link
Member

@jessegrabowski jessegrabowski commented May 23, 2025

Checking out how VI works with a simple from-scratch implemention


📚 Documentation preview 📚: https://pymc--7799.org.readthedocs.build/en/7799/

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@twiecki
Copy link
Member

twiecki commented May 24, 2025

CC @ferrine

Copy link

codecov bot commented Jun 8, 2025

Codecov Report

Attention: Patch coverage is 0% with 41 lines in your changes missing coverage. Please review.

Project coverage is 92.64%. Comparing base (c8d83ca) to head (b7aa31d).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
pymc/variational/autoguide.py 0.00% 41 Missing ⚠️

❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (50.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #7799      +/-   ##
==========================================
- Coverage   92.84%   92.64%   -0.21%     
==========================================
  Files         107      108       +1     
  Lines       18380    18422      +42     
==========================================
+ Hits        17065    17067       +2     
- Misses       1315     1355      +40     
Files with missing lines Coverage Δ
pymc/variational/autoguide.py 0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@theorashid
Copy link
Contributor

I've read the notebook. I like the API a lot. I'm a fan of the numpyro guide/autoguide. Nice work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants