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

[Unitree][cross] set make job num to CPU threads #1873

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

sktometometo
Copy link
Contributor

@sktometometo sktometometo commented Oct 13, 2023

Make job number of make to number of CPU threads to make it adaptive for CPU number and make it faster on a machine with many CPU (e.g. mars)

Differnces between this branch and master is below. This experiment is done on mars.

Duration to run build_ros1_dependencies.sh

master

real 22m32.596s
user 0m0.833s
sys 0m2.247s

this branch

time ./build_ros1_dependencies.sh

real 15m32.281s
user 0m0.940s
sys 0m2.088s

Duration to run build_ros1.sh

master

real 19m7.476s
user 0m2.100s
sys 0m1.676s

this branch

real 16m43.714s
user 0m2.018s
sys 0m1.586s

@sktometometo sktometometo marked this pull request as draft October 13, 2023 00:59
@sktometometo sktometometo changed the title [Unitree][cross] set make job num to CPU + 1 [Unitree][cross] set make job num to CPU threads Oct 13, 2023
@sktometometo sktometometo marked this pull request as ready for review October 13, 2023 01:20
@sktometometo sktometometo marked this pull request as draft October 13, 2023 02:02
@sktometometo
Copy link
Contributor Author

Got some error, revert to draft

@sktometometo sktometometo marked this pull request as ready for review October 15, 2023 13:11
@k-okada k-okada added the hacktoberfest-accepted https://hacktoberfest.digitalocean.com/hacktoberfest-update label Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted https://hacktoberfest.digitalocean.com/hacktoberfest-update Unitree
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants