Skip to content

Conversation

@Jorriss
Copy link
Owner

@Jorriss Jorriss commented Jun 4, 2025

ff47fed - Big changes to parsing and displaying parsing results

  • Refactored parsing and display logic.
  • Added new features, improved output formatting, and updated dependencies.
  • Large-scale code additions and deletions for modernization.
  • Improved language localization

b28728a - Moved display code into a module.

  • Modularized display logic by moving it into its own module.
  • Updated imports and file structure.
  • Cleaned up and reorganized codebase, including moving and renaming files.

9b0ca02 - Refactored the parsing engine to parse new-ish IO columns.

  • Major refactor of the parsing engine to support new IO columns.
  • Updated display and parser modules, language files, and main entry point.
  • Improved parsing logic and updated language support.

0fad233 - Renamed IO table css classes.

  • Refactored CSS class names related to IO tables for clarity and consistency.
  • Updated references in js modules and SCSS files.

2ecff48 - Added unit tests.

  • Added comprehensive unit tests for the application.
  • Created new test files for displaystats, language, parser, storage, and UI modules.
  • Updated README, package files, and configuration for testing.
  • Large addition of test code and supporting config.

Bug fixes:
#29 Clear the "Show Example Statistics IO Output" check-box when the text entry field is modified.
#30 If the "Show Example Statistics IO Output" check-box is unchecked the input text field is cleared.
#50 Bug in scan count and logical reads
#55 Invalid numbers for Scan count / Total logical
#56 Aggregating incorrectly
#60 Read-ahead-reads not reported
#61 Copy Results Button
#62 Add a Scroll To Top button at that bottom of the page.
#63 Add new-ish Statistics IO columns

Jorriss and others added 20 commits May 8, 2025 10:26
Truing up dev with master
Upgraded all modules to the current version, replaced them with newer modules, or removed them all together.
Includes: jQuery, Bootstrap, DataTables, Moment -> Luxon, Numeral (removed), Purl (removed), jQuery Cookies (removed).
Also reduced reliance on jQuery. Though we still need to to use DataTables.
Refactored the parser to separate the UI from the parsing data. This should assist in creating new features more easily.  Also separated logic into different modules. Introduced vite to assist in building the project.
Moved "Tables have scrollbars" into language files.
Had vite search and replace version number in html.
To be a bit more clear.
The Google Analytics script is in the json file .env.json. If the file doesn't exist the analytics script won't be added.
When loading a StatsIOInfo object if the value array is less than the columns array the load would continue and load undefined for the remaining columns. A check was added in the loop to check if the range was out of bounds, if it was the new StatsIOInfo object would be returned.

There are situations where the columns will mismatch from the values (i.e. Columnstore). We many need to parse each column in each line.
Also fixed the formatting number of table data. The international formatting was causing some of the columns to be treated as a string.
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