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.

Sign up for GitHub

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

Jump to bottom

[Perf] Implement WindowsMemoryResultParser, WindowsBatteryResultParser and refine WindowsMemoryInspector.ps1 #264

Merged
merged 13 commits into from
Feb 14, 2023
Merged

[Perf] Implement WindowsMemoryResultParser, WindowsBatteryResultParser and refine WindowsMemoryInspector.ps1 #264

merged 13 commits into from
Feb 14, 2023

Conversation

AC-Phoenix
Copy link
Collaborator

Changes

  • Implement WindowsMemoryResultParser, WindowsMemoryMetrics
  • Implement WindowsBatteryResultParser, WindowsBatteryMetrics
  • Refine WindowsMemoryInspector.ps1

Next

  • Need to verify if the agent configured with elevated privileges can bypass the UAC popup without changing the UAC configuration.
  • Add a new method in ShellUtils to run the admin command if the above one is not feasible.
  • Make memory & battery test work with more processes of other Windows apps.

@AC-Phoenix AC-Phoenix changed the title User/yazhang1/windows perf parser [Perf] Implement WindowsMemoryResultParser, WindowsBatteryResultParser and Refine WindowsMemoryInspector.ps1 Feb 13, 2023
@AC-Phoenix AC-Phoenix changed the title [Perf] Implement WindowsMemoryResultParser, WindowsBatteryResultParser and Refine WindowsMemoryInspector.ps1 [Perf] Implement WindowsMemoryResultParser, WindowsBatteryResultParser and refine WindowsMemoryInspector.ps1 Feb 13, 2023
@AC-Phoenix AC-Phoenix merged commit da425af into main Feb 14, 2023
@AC-Phoenix AC-Phoenix deleted the user/yazhang1/windows_perf_parser branch February 14, 2023 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@taoran6 taoran6 taoran6 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AC-Phoenix @taoran6 @YangZhang0219