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

Add _collect() to __init__()? #2

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

Conversation

Andrew-McNab-UK
Copy link

I needed to add this to get the mjf.py script to work from the command line. The function description does say that it both initializes and collects.

I needed to add this to get the mjf.py script to work from the command line. The function description does say that it both initializes and collects.
@Andrew-McNab-UK
Copy link
Author

Sorry, I now see you deliberately removed this a few months ago. Another option is to add it to _run() instead.

@roiser
Copy link
Owner

roiser commented May 5, 2014

I was already to reply, I have a switch called "ext" - lousy name for denoting the script is run from the command line - we could use this in the constructor to call collect if "ext". Thanks for the patch (the other one), I'll integrate them shortly.

cheers

Stefan

On 2 May 2014, at 18:13, Andrew McNab [email protected] wrote:

Sorry, I now see you deliberately removed this a few months ago. Another option is to add it to _run() instead.


Reply to this email directly or view it on GitHub.

Stefan Roiser, CERN, IT Dept, CH-1211 Geneva 23
tel: +41 76 4875334; aim, msn, gtalk, skype: sroiser

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