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

Python version should be mentioned explicitly while spawning subprocess #2

Open
gshrikant opened this issue Sep 28, 2017 · 4 comments

Comments

@gshrikant
Copy link

Subprocess calls in guineapig.py (line 941, for example) invoke a Python child process using just python. This should probably be python2 or python3 explicitly depending on the version expected by the script. If a user has python aliased to python3 this would result in a confusing error message.

@RyanLeiTaiwan
Copy link

Same issue! So far, I can only soft-link my python to python2 again on my MacOS system.

Adding alias python='python2' to ~/.bash_profile does not resolve the problem.

@wwcohen
Copy link
Member

wwcohen commented Feb 13, 2018

I just pushed a fix to this, in 1.3.7 - you can now specify pythonCommand in --opts or use the env variable GP_PYTHON_COMMAND

10-605/gpigtut-1.3.7.tgz has a copy of the tutorial if you

@RyanLeiTaiwan
Copy link

It works now! Thank you.
However, http://www.cs.cmu.edu/~wcohen/10-605/gpigtut-1.3.7.tgz cannot be extracted correctly:

$ tar xvf gpigtut-1.3.7.tgz 
x tutorial/marker.txt
x tutorial/guineapig.py
x tutorial/bluecorpus.txt
x tutorial/corpus.txt
x tutorial/idcorpus.txt
x tutorial/redcorpus.txt
x tutorial/wc-for-bluecorpus.txt
x tutorial/wc-for-redcorpus.txt
x tutorial/id-parks.txt
x tutorial/add-ids.py
x tutorial/combiner-wordcount.py
x tutorial/guineapig.py: Can't create 'tutorial/guineapig.py'
x tutorial/instance-wordcount.py
x tutorial/longer-wordcount.py
x tutorial/multi-wordcount-hadoop.py
x tutorial/multi-wordcount.py
x tutorial/ntup-wordcount.py
x tutorial/pagerank.py
x tutorial/param-wordcount.py
x tutorial/phirl-improved.py
x tutorial/phirl-naive.py
x tutorial/phirl-naive1_2.py
x tutorial/phirl-naive1_3.py
x tutorial/prefix-count.py
x tutorial/smallvoc-tfidf-simplified.py
x tutorial/smallvoc-tfidf.py
x tutorial/tfidf.py
x tutorial/wordcmp.py
x tutorial/wordcount.py
x tutorial/wordprob.py
tar: Error exit delayed from previous errors.

@wwcohen
Copy link
Member

wwcohen commented Feb 13, 2018 via email

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

No branches or pull requests

3 participants