Skip to content

[feat] Allow the -P option to redefine existing parameters#3554

Merged
vkarak merged 1 commit intoreframe-hpc:developfrom
vkarak:feat/reparameterize
Sep 24, 2025
Merged

[feat] Allow the -P option to redefine existing parameters#3554
vkarak merged 1 commit intoreframe-hpc:developfrom
vkarak:feat/reparameterize

Conversation

@vkarak
Copy link
Contributor

@vkarak vkarak commented Sep 17, 2025

To achieve this we add type information to parameters, so that the user-supplied arguments can be correctly converted by the test. The change is seamless, as by default no type information is required for parameters, but if users want to re-parameterize a test, then the parameters must have a type.

Closes #2674.

Todos

  • Update the docs

@codecov
Copy link

codecov bot commented Sep 17, 2025

Codecov Report

❌ Patch coverage is 93.10345% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.23%. Comparing base (df31923) to head (a32c19b).
⚠️ Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
reframe/frontend/testgenerators.py 92.45% 4 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #3554   +/-   ##
========================================
  Coverage    91.22%   91.23%           
========================================
  Files           62       62           
  Lines        13470    13508   +38     
========================================
+ Hits         12288    12324   +36     
- Misses        1182     1184    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

To achieve this we add type information to parameters, so that the
user-supplied arguments can be correctly converted by the test.
@vkarak vkarak force-pushed the feat/reparameterize branch from a464d9c to a32c19b Compare September 19, 2025 20:54
@vkarak vkarak changed the title Allow the -P option to redefine existing parameters [feat] Allow the -P option to redefine existing parameters Sep 23, 2025
@github-project-automation github-project-automation bot moved this from Todo to In Progress in ReFrame Backlog Sep 23, 2025
@vkarak vkarak merged commit 584c84b into reframe-hpc:develop Sep 24, 2025
39 checks passed
@vkarak vkarak deleted the feat/reparameterize branch September 24, 2025 22:23
@github-project-automation github-project-automation bot moved this from In Progress to Done in ReFrame Backlog Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Allow setting existing parameters from the command line

2 participants