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

Do we really need a config file any more? #613

Closed
dpryan79 opened this issue Oct 6, 2017 · 4 comments
Closed

Do we really need a config file any more? #613

dpryan79 opened this issue Oct 6, 2017 · 4 comments

Comments

@dpryan79
Copy link
Collaborator

dpryan79 commented Oct 6, 2017

At the moment, the config file only has two entries: default_proc_number and tmp_dir. tmp_dir can already be changed with environment variables. I wonder if anyone ever bothers to change default_proc_number. I would propose getting rid of this entirely and making the default number of threads 1. I kind of assume that people are already specifying how many threads they want to use already.

As an aside, Galaxy already does away with the config file.

@endrebak
Copy link

I'd remove it.

@bgruening
Copy link
Member

Me too :)

@vivekbhr
Copy link
Member

👍 for remove..

dpryan79 added a commit that referenced this issue Nov 12, 2017
@dpryan79
Copy link
Collaborator Author

This is now gone in develop. The default number of threads for all tools is 1, as seems reasonable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants