Skip to content

Initial Draft of APE16: CPU Threading - #46

Closed
jamienoss wants to merge 1 commit into
astropy:masterfrom
jamienoss:OpenMP-APE
Closed

jamienoss wants to merge 1 commit into
astropy:masterfrom
jamienoss:OpenMP-APE

Conversation

@jamienoss

Copy link
Copy Markdown

This still needs a lot of work. I've stalled on it a little and rather it sat out here than not at all.

Signed-off-by: James Noss [email protected]

@cdeil

cdeil commented Dec 6, 2018

Copy link
Copy Markdown
Member

FYI, in case you want to continue the APE: I asked for Scipy, and they don't use OpenMP because it's not considered portable enough. They use very little multi-core overall: mostly in the linear algebra packages they call into (don't know what those use), and very few functions use Python multiprocessing from the Python standard library, and even there because of issues on some platforms they are considering using something like https://github.com/joblib/joblib that's apparently more robust. I do think a short APE that is a summary of available options in Python and what other projects use would be very useful, as a basis for discussions what (if anything) to do in Astropy to improve performance in the future. Not sure if an APE would be the appropriate format for that, it could also be a wiki page, or a poster / proceeding at e.g. a Scipy conference "Options for multi-core CPU processing for scientific Python packages in 2019".

@eteq

eteq commented Dec 10, 2019

Copy link
Copy Markdown
Member

@jamienoss - are you still interested in pursuing this APE? If you don't think you have the time/desire, it may be we need to consider this APE withdrawn. We can then close this PR, but then you can leave your branch in place so that if anyone wants to they can pick up this topic (but include you as a co-author of course if they start from that text!)

@jamienoss jamienoss closed this Feb 3, 2020
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.

3 participants