Skip to content

#73 New data model#88

Merged
jamesmarkchan merged 4 commits intodevfrom
new-data-model
Sep 15, 2025
Merged

#73 New data model#88
jamesmarkchan merged 4 commits intodevfrom
new-data-model

Conversation

@jamesmarkchan
Copy link
Copy Markdown
Member

@jamesmarkchan jamesmarkchan commented Sep 15, 2025

summary of changes:

  1. refactored benchmark data model to separate benchmark from operation
    • jpa references updated, cascade all
    • Parent class is Benchmark, BenchmarkOperation is the child class
    • allows more complex modeling like crystal disk mark models
  2. keyboard can now be used to select a new record in the benchmark operation history will automatically load the record
  3. Benchmark tab renamed to Benchmark Operations
  4. introduced benchmark type which translates to IOMode at the operation level
  5. fixed updating of num threads and Benchmark Type on load

@jamesmarkchan jamesmarkchan requested review from a team and tylermlui September 15, 2025 02:58
@jamesmarkchan jamesmarkchan self-assigned this Sep 15, 2025
@jamesmarkchan jamesmarkchan changed the base branch from main to dev September 15, 2025 04:43
@vpstackhub
Copy link
Copy Markdown
Contributor

Tested on Windows 11 -Branch: new-data-model
I successfully cloned the updated repo and checked out the new-data-model branch. After building the project in NetBeans, the UI launched cleanly with no errors or build issues.
Attaching screenshot of the running interface for confirmation.
Everything looks great from my side-no regressions or crashes observed. Ready for merge!

Screenshot 2025-09-15 093220

@vpstackhub
Copy link
Copy Markdown
Contributor

Screenshot 2025-09-15 093149

@jamesmarkchan jamesmarkchan removed the request for review from tylermlui September 15, 2025 17:27
@jamesmarkchan jamesmarkchan merged commit fa51a8b into dev Sep 15, 2025
jamesmarkchan added a commit that referenced this pull request Dec 17, 2025
@jamesmarkchan jamesmarkchan deleted the new-data-model branch December 27, 2025 00:25
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