ICB-DCM / pyABC

distributed, likelihood-free inference
https://pyabc.rtfd.io
BSD 3-Clause "New" or "Revised" License
205 stars 44 forks source link

Replace documentation references to SimpleModel with FunctionModel #629

Closed omsai closed 8 months ago

omsai commented 9 months ago

SimpleModel was removed in 3fc1a52c5d124f62add2cf7b245fb582ce2d6cea

flake8: install_deps> python -I -m pip install 'black>=22.3.0' 'flake8-bandit>=4.1.1' 'flake8-black>=0.2.3' 'flake8-bugbear>=22.8.23' 'flake8-colors>=0.1.6' 'flake8-comprehensions>=3.2.3' 'flake8-isort>=4.0.0' 'flake8-print>=5.0.0' 'flake8>=3.8.3' flake8: commands[0]> flake8 pyabc test test_performance setup.py Unable to find qualified name for module: setup.py pyabc/acceptor/init.py:8:1: BLK100 Black would make changes. pyabc/acceptor/acceptor.py:443:27: BLK100 Black would make changes. pyabc/distance/distance.py:72:13: BLK100 Black would make changes. pyabc/distance/kernel.py:2:1: BLK100 Black would make changes. pyabc/distance/scale.py:34:1: BLK100 Black would make changes. pyabc/predictor/predictor.py:484:13: BLK100 Black would make changes. pyabc/storage/migrations/versions/1_20210219_add_particles_proposal_id.py:8:1: BLK100 Black would make changes. pyabc/util/par_trafo.py:97:13: BLK100 Black would make changes. test/base/test_samplers.py:231:69: BLK100 Black would make changes. test_performance/test_samplerperf.py:163:69: BLK100 Black would make changes. flake8: exit 1 (3.28 seconds) /Users/pnanda/src/pyABC> flake8 pyabc test test_performance setup.py pid=71418 flake8: FAIL code 1 (6.92=setup[3.64]+cmd[3.28] seconds) evaluation failed :( (6.96 seconds)


- The above Black-related issues don't flag the `pyabc/model/model.py` that I changed, so I uninstalled the commit hook to create the PR.
codecov-commenter commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.65%. Comparing base (6446a77) to head (0d1305b). Report is 235 commits behind head on develop.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #629 +/- ## =========================================== - Coverage 84.08% 81.65% -2.44% =========================================== Files 91 141 +50 Lines 5260 8535 +3275 =========================================== + Hits 4423 6969 +2546 - Misses 837 1566 +729 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

stephanmg commented 8 months ago

Thanks for the fix.