Skip to content

Conversation

Armavica
Copy link
Member

This PR closes #6124 by adding an example and explanation of pm.Deterministic.

It also repairs the plot of pm.Interpolated which was not working, and fixes some of the doc makefile commands.

Checklist

Major / Breaking Changes

  • ...

Bugfixes / New features

  • ...

Docs / Maintenance

  • Fix make serve
  • Fix make html
  • Fix make clean
  • Repair the plot of pm.Interpolated
  • Add an explanation and example to pm.Deterministic

@codecov
Copy link

codecov bot commented Sep 13, 2022

Codecov Report

Merging #6126 (c032d6e) into main (bbb3082) will decrease coverage by 0.48%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6126      +/-   ##
==========================================
- Coverage   91.02%   90.54%   -0.49%     
==========================================
  Files          99      102       +3     
  Lines       20531    21241     +710     
==========================================
+ Hits        18689    19233     +544     
- Misses       1842     2008     +166     
Impacted Files Coverage Δ
pymc/distributions/continuous.py 97.51% <ø> (ø)
pymc/distributions/transforms.py 100.00% <ø> (ø)
pymc/model.py 88.23% <ø> (ø)
pymc/variational/test_functions.py 38.09% <0.00%> (-61.91%) ⬇️
pymc/variational/stein.py 48.33% <0.00%> (-51.67%) ⬇️
pymc/variational/operators.py 57.14% <0.00%> (-35.72%) ⬇️
pymc/variational/inference.py 50.77% <0.00%> (-35.32%) ⬇️
pymc/variational/approximations.py 54.79% <0.00%> (-32.53%) ⬇️
pymc/variational/callbacks.py 78.00% <0.00%> (-18.00%) ⬇️
pymc/variational/opvi.py 75.20% <0.00%> (-11.85%) ⬇️
... and 19 more

@ricardoV94
Copy link
Member

My review concerns the changes in docstrings, not the changes to the makerfile

@Armavica Armavica added the docs label Sep 14, 2022
@ricardoV94 ricardoV94 merged commit 78dd40f into pymc-devs:main Sep 23, 2022
@Armavica Armavica deleted the cleanup-docs branch October 7, 2024 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The docs for deterministic should include an example

3 participants