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

Improve error handling & add some diagnostic output #61

Merged
merged 12 commits into from
Mar 17, 2024
Merged

Improve error handling & add some diagnostic output #61

merged 12 commits into from
Mar 17, 2024

Conversation

Vaiz
Copy link
Contributor

@Vaiz Vaiz commented Mar 17, 2024

Hi tigros,
I've introduced some enhancements to streamline the debugging process in this pull request. I hope you'll find these changes beneficial, or at least some of them.

Changes:

  • Added error message if main.exe encounters errors. It includes the output from the application.
  • Added exception for cases where totmem is less than usedMem.
  • Added exception if no performance counters are found.
  • Disabled the goButton until performance counters are initialized. Initialization takes quite some time, and I managed to start transcription before counters were initialized.
  • Renamed certain elements to better reflect their roles and responsibilities.

#60

@tigros tigros merged commit 80bcae4 into tigros:main Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants