Skip to content

CI test Arviz compat #4258

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

Merged
merged 4 commits into from
Nov 26, 2020
Merged

CI test Arviz compat #4258

merged 4 commits into from
Nov 26, 2020

Conversation

MarcoGorelli
Copy link
Contributor

closes #4256

@MarcoGorelli MarcoGorelli changed the title CI testArviz compat CI test Arviz compat Nov 26, 2020
pymc3/tests/test_updates.py
pymc3/tests/test_variational_inference.py
- |
pymc3/tests/test_distributions.py
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this file alone takes > 10 minutes! 😱

@codecov
Copy link

codecov bot commented Nov 26, 2020

Codecov Report

Merging #4258 (0ac4ee1) into master (2723b6c) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4258      +/-   ##
==========================================
+ Coverage   88.14%   88.15%   +0.01%     
==========================================
  Files          87       87              
  Lines       14243    14243              
==========================================
+ Hits        12554    12556       +2     
+ Misses       1689     1687       -2     
Impacted Files Coverage Δ
pymc3/step_methods/hmc/base_hmc.py 97.50% <0.00%> (-0.84%) ⬇️
pymc3/parallel_sampling.py 89.30% <0.00%> (+0.94%) ⬆️

@twiecki twiecki merged commit a6295a3 into pymc-devs:master Nov 26, 2020
@twiecki
Copy link
Member

twiecki commented Nov 26, 2020

Thanks @MarcoGorelli!

@MarcoGorelli MarcoGorelli deleted the arviz-compat branch November 26, 2020 11:33
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.

Run arviz compatibility job in GitHub Actions
2 participants