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

Automatically sets the default number of parallel rendering threads #193

Closed
lc-soft opened this issue Dec 23, 2019 · 3 comments
Closed

Automatically sets the default number of parallel rendering threads #193

lc-soft opened this issue Dec 23, 2019 · 3 comments
Labels
easy This issue is easy to solve help wanted

Comments

@lc-soft
Copy link
Owner

lc-soft commented Dec 23, 2019

Issuehunt badges

Is your feature request related to a problem? Please describe.

Changing the number of parallel threads requires recompiling the source code, which is too much trouble.

Describe the solution you'd like

LCUI/src/display.c

Lines 55 to 64 in 949db38

#ifdef USE_OPENMP
/**
* Parallel rendering threads
* We recommend that you set it to half the number of CPU logical cores
*/
#define PARALLEL_RENDERING_THREADS 4
#else
#define PARALLEL_RENDERING_THREADS 1
#endif

Describe alternatives you've considered
None.

Additional context
Please wait for issue #191 to close before work on this issue.


IssueHunt Summary

Backers (Total: $0.00)

Become a backer now!

Or submit a pull request to get the deposits!

Tips


IssueHunt has been backed by the following sponsors. Become a sponsor

@lc-soft lc-soft changed the title Automatically sets the default number of parallel render threads Automatically sets the default number of parallel rendering threads Dec 23, 2019
@lc-soft lc-soft added easy This issue is easy to solve help wanted labels Dec 23, 2019
@lc-soft lc-soft pinned this issue Dec 23, 2019
@issuehunt-oss
Copy link

issuehunt-oss bot commented Dec 23, 2019

@lc-soft has funded $5.00 to this issue.


@issuehunt-oss issuehunt-oss bot added the 💵 Funded on Issuehunt This issue has been funded on Issuehunt label Dec 23, 2019
@lc-soft
Copy link
Owner Author

lc-soft commented Feb 26, 2020

No need to do

@lc-soft lc-soft closed this as completed Feb 26, 2020
@issuehunt-oss
Copy link

issuehunt-oss bot commented Feb 26, 2020

@lc-soft has cancelled funding for this issue.(Cancelled amount: $5.00) See it on IssueHunt

@issuehunt-oss issuehunt-oss bot removed the 💵 Funded on Issuehunt This issue has been funded on Issuehunt label Feb 26, 2020
@lc-soft lc-soft unpinned this issue Mar 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
easy This issue is easy to solve help wanted
Projects
None yet
Development

No branches or pull requests

1 participant