voichek / kmersGWAS

A library for running k-mers based GWAS
GNU General Public License v3.0
100 stars 24 forks source link

environmental python to execute scripts #51

Closed BenjaminSchwessinger closed 4 years ago

BenjaminSchwessinger commented 4 years ago

It might be good to include #! /usr/bin/env python on top of your python scripts as otherwise they tend you use the system python which is likely not to be python 2.7 anymore.

Thanks! Benjamin

voichek commented 4 years ago

Thanks for the comment!

I was under the impression that if you use: python2.7 kmers_gwas.py ... it will use python2.7, no? Even if you specify the version in the command line explicitly it might use a different version?

Best, Yoav

BenjaminSchwessinger commented 4 years ago

You are correct if you use python2.7 in front of your script then all is fine.

I was putting things in the path/bin and got some error messages hence the comment. I fixed it up. Just in case others run into the same problem.

Thanks! Testing on E.coli now before digging in. Looking forward to it.

Thanks for sharing and support.

On 24/7/20 6:14 pm, Yoav Voichek wrote:

Thanks for the comment!

I was under the impression that if you use: |python2.7 kmers_gwas.py ...| it will use python2.7, no? Even if you specify the version in the command line explicitly it might use a different version?

Best, Yoav

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/voichek/kmersGWAS/issues/51#issuecomment-663402783, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRMZBWUIJIHSV2PG5KQ3OTR5E7E7ANCNFSM4PGO2Y5A.

--

Benjamin Schwessinger PhD. ARC Future Fellow 2018 Senior Lecturer Division of Plant Science Research School of Biology College of Science Linnaeus Building (134), Linnaeus Way The Australian National University Canberra ACT 0200 Australia

P: +61 2 6125 7794 (Office, 3.085) benjamin.schwessinger@anu.edu.au lab webpage http://teamschwessinger.com/, @schwessinger http://twitter.com/schwessinger, blog http://blushgreengrassatafridayafternoon.wordpress.com/, google scholar http://scholar.google.com.au/citations?user=lEhYW3QAAAAJ&hl=en

Ambassador of ASAPbio <:"https://asapbio.org//"> Member of eLife's Early Career Advisory Group <:"https://elifesciences.org/about/people/early-career"> Co-founder of Reproducibility 4 Everyone <:"https://www.repro4everyone.org/">

CRICOS Provider # 00120C

voichek commented 4 years ago

Thanks for letting me know, and on the possible solution. I added it to my list of things to improve/test for the next version.

Hope the next steps will go smooth, Yoav