search

pymc-devs / pymc4

Experimental PyMC interface for TensorFlow Probability. Official work on this project has been discontinued.
Apache License 2.0
712 stars 113 forks source link

Centralize fixtures #194

Open lucianopaz opened 4 years ago

lucianopaz commented 4 years ago

As pymc4 grows, the test suite gets bigger and we add more fixtures to the test scripts. I think that it would be good practice to start to move the fixtures out of the test scripts and into either a central conf.py or separate modules that get imported into a conf.py. Some fixtures are awfully similar across test files and we may be able to fuse them into a single coherent fixture.

lucianopaz commented 4 years ago

Reopening because #238 only moved the fixtures from test sampling to conftest.py, but the rest of the test scripts have fixtures too. They should all be moved and potentially refactored to reduce duplicate code.