-
Notifications
You must be signed in to change notification settings - Fork 49
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
(TX2 Fix) Updating gtop to use command-line arguments #5
base: master
Are you sure you want to change the base?
Conversation
…, not guess based on fields out of tegrastats. Added EMC plot.
Hi @zlacelle! Awesome work! Could you check if it is working on TX2? Cheers, |
@confusedrobot if you can send me a few lines of tegrastats output, I will run it through the code today and make sure it's good. I'll try to add in some sanity checks as well. There are a few other changes I made in my repo which I'll also push, once I clean them up. |
Here's my tegrastats output: |
@zlacelle let me know if there's anything else I can provide to help debug this. |
@confusedrobot Just didn't have time today, I'll hopefully have some time to take a look tomorrow. |
Do you get it working till tomorrow? Would be super awesome and important for me! |
I created a new pull request here: #6 I switched to what should be a more robust parsing method. @confusedrobot I checked your tegrastats output, and it matches mine, so it should work. If it doesn't work, compile with debug flags and send a core dump. @gustavz If it doesn't work for you, send me your tegrastats output. Right now, the only testing I can do is on my TX2 running JetPack 3.2 |
@zlacelle It works!! But would it be possible that the tool averages the usage over a range of 30 secs or something? |
@zlacelle sorry it took awhile to respond, I am going to try the new fix now. |
Hello @confusedrobot, @gustavz and @zlacelle ! I am planning to add test suite for supported devices so new pull requests can be tested against it. For this I would need an example output from Thank you, |
@martinkersner Will do. I'll try to get it from a Xavier and a TX2, but might take a few days. |
@zlacelle Great! Thank you. |
to specify the board type, not guess based on fields out of tegrastats. Added EMC plot.