Skip to content

Add support for global and test-local memory overrides#299

Draft
zack-rma wants to merge 1 commit into
mainfrom
bugfix/compute-memory
Draft

Add support for global and test-local memory overrides#299
zack-rma wants to merge 1 commit into
mainfrom
bugfix/compute-memory

Conversation

@zack-rma

@zack-rma zack-rma commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Description

Adds global and test-local memory bounds overrides. Includes unit test for property parser.

Motivation and Context

Addresses #208

Types of changes

Yes/No Pull Request Type Description
Yes Bug fix non-breaking change which fixes an issue
New feature non-breaking change which adds functionality
Breaking change fix or feature that would cause existing functionality to change
Documentation change non-breaking change which modifies or updates documentation
New tests new unit tests, test scenarios, or test case documentation
Triggers regression testing change affects downstream modules and will require regression testing
Other

How Has This Been Tested?

Unit tested, local testing TBD

Notes for Reviewers

Please consider the following when reviewing this PR:

  • Correctness: Does the code do what it claims? Are edge cases handled appropriately?
  • Clarity: Is the code readable, maintainable and aligned with SOLID design principles?
  • Impact: Will this change affect other parts of the system? Any potential regressions?
  • Testing: Are the test cases sufficient and appropriate? Are there gaps in coverage?
  • Documentation: Does this require updates to code comments, README, or other docs?

@sonarqubecloud

sonarqubecloud Bot commented Jun 5, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@zack-rma zack-rma moved this to In Progress in WRIMS 3 Development Jun 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant