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

[806] Add System Profiler Utility #2224

Merged
merged 7 commits into from
May 14, 2024

Conversation

Rixxan
Copy link
Contributor

@Rixxan Rixxan commented May 2, 2024

Description

When receiving bug reports, there is a number of details about the user's environment that need collected. These include the version of EDMC, the OS version and architecture, the game version, and other diagnostic details.

This PR adds a standalone system utility to gather all of this information in one place, as well as adding links to the utility from the Help menu of EDMC. The profiler uses only standard library tools we already bundle with EDMC, and minimal external utilities also present in EDMC. Some aspects of the system profiler are imported, such as config and Monitor, so any issues with those systems may not be diagnosable by this tool (though that will tell us things too!)

The Profiler can be run as an EXE bundled with default distributions, or by the console if running from source with the --out-console argument.

This PR also significantly reorders and cleans up the default bug report template, removing old guidance that has not been "new" for years.

Finally, this PR optimizes several of the repository images, reducing image file sizes by 25%

Type of Change

  • Feature Enhancement

Screenshots

327233323-57fae736-6cab-44ca-bc36-6987a15c909a

How Tested

Tested under a number of conditions to ensure uniform output, including intentionally triggering failures in the ability to pull EDMC or Journal data. Tested in compiled and source builds.

Notes

Resolves #806

Uses the new translation system

@Rixxan Rixxan added enhancement devtools Developer tools related python Pull requests that update Python code New User Experience ui User Interface labels May 2, 2024
@Rixxan Rixxan self-assigned this May 2, 2024
@Rixxan Rixxan linked an issue May 2, 2024 that may be closed by this pull request
@Rixxan Rixxan added this to the 5.11.0 milestone May 6, 2024
ImgBotApp and others added 2 commits May 6, 2024 13:53
*Total -- 146.57kb -> 109.96kb (24.98%)

/img/UI-Athanasius-101-odyssey-docked-onfoot-cooled-down-l.png -- 6.88kb -> 3.53kb (48.69%)
/img/UI-annotated.png -- 7.69kb -> 4.40kb (42.74%)
/img/win.png -- 6.74kb -> 4.75kb (29.58%)
/img/win_dark.png -- 5.26kb -> 3.87kb (26.49%)
/img/win_transparent.png -- 71.73kb -> 53.46kb (25.47%)
/img/Balance.png -- 48.27kb -> 39.95kb (17.24%)

Signed-off-by: ImgBotApp <[email protected]>
@Rixxan Rixxan marked this pull request as ready for review May 11, 2024 22:19
Copy link
Member

@C1701D C1701D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@C1701D C1701D merged commit 6969eec into EDCD:develop May 14, 2024
3 checks passed
@C1701D C1701D deleted the enhancement/806/system-profiler branch May 14, 2024 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devtools Developer tools related enhancement New User Experience python Pull requests that update Python code ui User Interface
Projects
Development

Successfully merging this pull request may close these issues.

Small utility to report on OS version and other useful information
3 participants