Updated to version 1.14.0 - #19
Merged
Merged
Conversation
orestiskapar
previously requested changes
Sep 17, 2026
btsouts
approved these changes
Sep 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request includes several improvements and clarifications across the benchmarking automation tool, its documentation, and CI configuration. The most significant changes are the simplification and clarification of Intel PIN support (making it optional and controlled by a new flag), updates to the README for better user guidance, and enhancements to the GitHub workflow and dependencies.
Intel PIN integration and documentation improvements:
Intel PIN is now optional and controlled by a
--measure-dynamic-instructionsflag instead of a--path-to-pinargument. The tool only uses PIN if this flag is set andPIN_ROOTis configured; otherwise, PIN is never invoked. Documentation, argument parsing, and code references have been updated to reflect this change. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]The README and help text now clarify that Intel PIN is off by default, and describe exactly when and how it is used, including error handling and reporting of missing instruction counts. [1] [2] [3] [4] [5] [6] [7]
Documentation and usability enhancements:
The README includes improved installation instructions, clarifies optional dependencies (such as Google Sheets integration), and provides more detailed usage examples for plotting and sampling from Ux Data. [1] [2] [3]
The section on benchmarking with UxHw and Intel PIN has been rewritten for clarity, and the Pin tool is now described as optional. [1] [2]
Continuous Integration and dependency management:
The GitHub Actions workflow now checks that all submodules are up to date before proceeding, helping to prevent issues from outdated dependencies.
The Python version requirement in
pyproject.tomlis now explicitly capped at<4for better compatibility with future Python releases.Codebase and submodule updates:
The
assetssubmodule has been updated to a new commit.Internal code improvements include importing
importlib.utilfor runtime dependency checks, and defining a tuple of required modules for the optional Google Sheets integration. [1] [2]Other changes: