Skip to content
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

added setup.py to install think*. import via 'import thinkbayes' #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

johndavidmiller
Copy link

Hi Allen:

I really like using your think*.py code for quick prototyping and testing, esp. via ipython notebook, so I created a simple setup.py to install it. Hardly rocket science, but it makes it possible to use the code from any directory via "import thinkbayes"

Thanks for your great books! I've made them required reading for all my team.

-- jdm
John David Miller
Principal Research Data Scientist
Intel Corporation

@AllenDowney
Copy link
Owner

This is a good idea. Thanks!

Can I ask you to check something before I merge? I don't think you need both thinkbayes.py and thinkstats.py (In fact, I'm not sure thinkstats should be in this repo).

Would you check, and if so, remove the dependency on thinkstats?

@johndavidmiller
Copy link
Author

As I recall it had some goodness and I didnt see any harm in including it. Note that I made them separate imports, for better or worse, rather than a single import of the lot.

Fwiw thinkstats2.py from that book looks like a cross between thinkbayes.py and this thinkstats.py

-- jdm

On Mar 24, 2016, at 6:41 AM, Allen Downey [email protected] wrote:

This is a good idea. Thanks!

Can I ask you to check something before I merge? I don't think you need both thinkbayes.py and thinkstats.py (In fact, I'm not sure thinkstats should be in this repo).

Would you check, and if so, remove the dependency on thinkstats?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

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.

2 participants