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

Monitoring by application gives wrong values #61

Closed
gcorrall opened this issue Jul 10, 2024 · 2 comments
Closed

Monitoring by application gives wrong values #61

gcorrall opened this issue Jul 10, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@gcorrall
Copy link
Contributor

When monitoring an application by its name I think the output is wrong when there are a range of PIDs to process, If, for example, I run powerjoular -ta firefox the 'CPU Utilization Percentage' will permanently stay at 0.00%.

Looking at the code, I think there may be an error in cpu_stat_app.adb, and that only the first PID is processed; that is, in Update_PID_Array:

App_Data.PID_Array(Loop_I) := Integer'Value (String_Split.Slice (Subs, 1));

should be

App_Data.PID_Array(Loop_I) := Integer'Value (String_Split.Slice (Subs, I));

@adelnoureddine adelnoureddine self-assigned this Jul 11, 2024
@adelnoureddine adelnoureddine added the bug Something isn't working label Jul 11, 2024
@adelnoureddine
Copy link
Member

Thanks for the report, I'll look over it and add the fix.

@adelnoureddine
Copy link
Member

Fixed in 7ed27bf. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants