Skip to content

Conversation

michaelosthege
Copy link
Member

What is this PR about?
Removing the MLDA step method and related tests as it's moving to pymc-experimental in pymc-devs/pymc-extras#57.

Closes #5942 (moving)
Closes #4267 (no longer applies)

Checklist

Major / Breaking Changes

  • The MLDA step method was removed and is now available through pymc_experimental.step_methods.mlda. See pymc-experimental PR #57.

Bugfixes / New features

None

Docs / Maintenance

None

@codecov
Copy link

codecov bot commented Jul 24, 2022

Codecov Report

Merging #6007 (0c2a4cf) into main (f9e0dfd) will decrease coverage by 0.21%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6007      +/-   ##
==========================================
- Coverage   89.36%   89.14%   -0.22%     
==========================================
  Files          73       72       -1     
  Lines       13253    12892     -361     
==========================================
- Hits        11843    11493     -350     
+ Misses       1410     1399      -11     
Impacted Files Coverage Δ
pymc/step_methods/__init__.py 100.00% <ø> (ø)
pymc/backends/base.py 86.44% <0.00%> (-0.74%) ⬇️

@twiecki twiecki merged commit 1101818 into main Jul 24, 2022
@twiecki twiecki deleted the remove-mlda branch July 24, 2022 16:01
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.

Move MLDA stepper to pymc-experimental pymc3/tests/test_step.py::TestMLDA::test_acceptance_rate_against_coarseness fails for unknown reason

2 participants