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

A small fix #4

Closed
wants to merge 2 commits into from
Closed

A small fix #4

wants to merge 2 commits into from

Conversation

Guillawme
Copy link

Hi!

First of all, thanks a lot for this nice script (I saw your recent post on CCP4BB), it will really turn painless a significant part of my PhD thesis writing (which shall begin in a few months...). 😃

I suggest a little fix (see commit 0cc8636) that makes the script work under any python setup by just calling it from the shell command prompt.

As provided before this commit, the script could only (?) be run by typing the command

$ python /path/to/pycorn.py

Now, provided that /path/to is in the user's $PATH and pycorn.py is executable, the script can simply be called by typing

$ pycorn.py

Tested under Mac OS X 10.10.1 with Apple's default python setup. This should behave the same under any bash-like shell on any OS having a proper python setup.

Commit 5d028b8 only fixes a typo in console output (or what I saw as a typo, being non native English speaker...), and makes uniform the use of the term "plotting" instead of "printing" found in another console output (only a fussy reflex from someone who proofread a couple books in the past...).

Please don't ask me to propose a more elaborate pull request: I'm totally illiterate in programming. 😶

@pyahmed
Copy link
Owner

pyahmed commented Jan 17, 2015

Thank you for the comments.
I've changed the "Plotting" to "Printing", regarding the /usr/bin/python thingy - I added something similar in the dev-branch yesterday. But your suggestion seems more general, thank you it is implemented :-)

The "Alles safe - go go go" is a silly insider-joke and is intentionally mixing german/english ;-)

@Guillawme
Copy link
Author

Great! I guess I can now close this pull request.

@Guillawme Guillawme closed this Jan 17, 2015
@Guillawme Guillawme deleted the smallfix branch March 8, 2015 09:26
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.

None yet

2 participants