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

Consider changing PyTables default number of threads #25

Open
EricR86 opened this issue Sep 22, 2016 · 4 comments
Open

Consider changing PyTables default number of threads #25

EricR86 opened this issue Sep 22, 2016 · 4 comments
Labels
enhancement New feature or request minor

Comments

@EricR86
Copy link
Member

EricR86 commented Sep 22, 2016

Original report (archived issue) by Coby Viner (Bitbucket: cviner2, GitHub: cviner).


PyTables defaults to using 4 threads. This could pose a problem for cluster usage and could also be made more efficient when more slots are available.

It may be worthwhile for Genomedata to override the default values of tables.parameters.MAX_NUMEXPR_THREADS and tables.parameters.MAX_BLOSC_THREADS.

If not, this behaviour is likely something that Genomedata should explicitly document.

@EricR86
Copy link
Member Author

EricR86 commented Oct 11, 2016

Original comment by Michael Hoffman (Bitbucket: hoffman, GitHub: michaelmhoffman).


@EricR86 Can you please document this in Genomedata? I don't think there needs to be an option to set this within Genomedata. Client programs can change directly using the tables interface if they need to.

segway-task should also set both of these to one since there is an assumption that one Segway task = one thread. I don't think any other part of Segway or Segtools should be modified, for now. Then you can close this.

@EricR86
Copy link
Member Author

EricR86 commented Oct 13, 2016

Original comment by Eric Roberts (Bitbucket: ericr86, GitHub: ericr86).


@michaelmhoffman If there doesn't need to be an option set to this within Genomedata, why document this particular PyTables interface feature? There are plenty of other PyTables parameters that could be documented as well.

@EricR86
Copy link
Member Author

EricR86 commented Oct 16, 2016

Original comment by Michael Hoffman (Bitbucket: hoffman, GitHub: michaelmhoffman).


Many people will be in an environment where they assume one thread per process unless they do something to change it. And they might violate, say, a cluster use policy to declare the number of threads or cores being used, if they aren't made aware of this.

@EricR86
Copy link
Member Author

EricR86 commented Jul 6, 2019

Original comment by Coby Viner (Bitbucket: cviner2, GitHub: cviner).


  • changed state from "new" to "open"

@EricR86 EricR86 added minor enhancement New feature or request labels Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request minor
Projects
None yet
Development

No branches or pull requests

1 participant