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

make clear to use python2 #82

Closed
wants to merge 1 commit into from
Closed

make clear to use python2 #82

wants to merge 1 commit into from

Conversation

jannes-m
Copy link

Dear Jeroen,
thanks for this interesting book which I will read over Christmas! When compiling the book using bookdown, I had to explicitly state that we want to use Python2. The default on my machine is Python3, and I guess this is the case for most other users as well, and if this is not already the case, it will be in the coming years. Hence, using Python2 is just a quick-and-dirty fix, maybe the better idea is to rewrite the code snippet using Python3. Should I do that?

@jeroenjanssens
Copy link
Owner

Especially by now, rewriting the code snippet in Python 3 is the better option. I would be very grateful if you were to rewrite the snippet, although I can also fully understand that you've moved on in the meantime.

@andres-lowrie
Copy link

andres-lowrie commented Nov 20, 2020

I opened up this one: #107

on another note, @jeroenjanssens loved the first book looking forward to the next one !!! Would be happy to help triage issues on the repos if you're looking for a hand with that

@jannes-m jannes-m closed this by deleting the head repository Mar 24, 2023
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.

3 participants