-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add an option to limit the CPU usage #2
Comments
Hi, Thank you for posting this issue. May I ask what is the situation of adding this feature to limit CPU usage? |
In the slides of project 1, the instructor suggests discussing the running time. I think it is reasonable to fix the number of cores over the experiments. However, I may run other programs at the same time due to research or other courses. In this case, I can't expect the numbers of cores using by different runs are the same. |
I see. Thanks for the suggestions. I am not sure whether we should add this. After all, it's not very common except for teaching purposes. |
for any timing exp it's better to use a machine without other jobs..
…On 2020-03-07 21:41, djshen wrote:
In the slides of project 1, the instructor suggests discussing the
running time. I think it is reasonable to fix the number of cores over
the experiments. However, I may run other programs at the same time
due to research or other courses. In this case, I can't expect the
numbers of cores using by different runs are the same.
If this is not a common situation, please just ignore the feature
request. Thank you!
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2]. [ { ***@***.***": "http://schema.org", ***@***.***": "EmailMessage",
"potentialAction": { ***@***.***": "ViewAction", "target":
"#2?email_source=notifications\u0026email_token=ABI3BHW7KS4RSLFQR5TP3ELRGJFKHA5CNFSM4LDMGJYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEODZXQQ#issuecomment-596089794",
"url":
"#2?email_source=notifications\u0026email_token=ABI3BHW7KS4RSLFQR5TP3ELRGJFKHA5CNFSM4LDMGJYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEODZXQQ#issuecomment-596089794",
"name": "View Issue" }, "description": "View this Issue on GitHub",
"publisher": { ***@***.***": "Organization", "name": "GitHub", "url":
"https://github.com" } } ]
Links:
------
[1]
#2?email_source=notifications&email_token=ABI3BHW7KS4RSLFQR5TP3ELRGJFKHA5CNFSM4LDMGJYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEODZXQQ#issuecomment-596089794
[2]
https://github.com/notifications/unsubscribe-auth/ABI3BHTX3ASGMFV4CTNGKIDRGJFKHANCNFSM4LDMGJYA
|
OK. I agree that it is better to keep this project as simple as possible. |
I find that in project 3 we may need this feature to limit tensorflow to use single thread. |
We can add a command line option to adjust the CPU usage, so that the program won't use up all of the resources. In addition, one can compare the timing of different settings.
The text was updated successfully, but these errors were encountered: