search

lmc2179 / bayesian_bootstrap

bayesian bootstrapping in python
MIT License
121 stars 17 forks source link

Simplify Importing #19

Closed genos closed 2 years ago

genos commented 3 years ago

Hello again! Thanks as ever for this very valuable tool, it's been great.

In using it, I've found myself (a very lazy user 😆) wanting to simplify the importing structure a bit; hence, this PR.

I've moved bootstrap.py to __init__.py; this means that rather than requiring from bayesian_bootstrap.bootstrap import X for imports we only need to say from bayesian_bootstrap import X, simplifying things a little. I've updated the README, demos, and tests to reflect this change. I also tweaked the delta value for one test, as it failed intermittently with randomized data.

genos commented 2 years ago

Just checking in; any thoughts on this PR?

genos commented 2 years ago

Happy new year! Any update on whether I should scrap this PR or not?

lmc2179 commented 2 years ago

Hi! Thanks for your patience...lots going on these days. I'll test this and get back to you!

lmc2179 commented 2 years ago

Looks good. Thank you!

genos commented 2 years ago

Great, thanks! Might I further presume upon your generosity and ask you to bump the version and update PyPI? 🙏