Skip to content

Conversation

@JavierCladellas
Copy link
Collaborator

The users can now provide a different json file for plot configuration, using the --plots-config parameter.
Previous files are still valid: i.e. users can choose to pass the plots config on the same file as the benchmark config (as before).

Main changes:

  • ConfigReader (benchmarking/reframe/config/configReader.py) now also accepts a list of filepaths. It will merge the json files into a single python dictionary and then perform data validation.
  • Added a --plots-config parameter to the execute-benchmark cli.
  • Splitted the plots section of configs into a different file.
  • Made plots field optional.

@JavierCladellas JavierCladellas added the enhancement A new feature or request label Nov 14, 2024
@JavierCladellas JavierCladellas self-assigned this Nov 14, 2024
@JavierCladellas JavierCladellas linked an issue Nov 14, 2024 that may be closed by this pull request
@netlify
Copy link

netlify bot commented Nov 14, 2024

Deploy Preview for benchmarking-polite-crostata-92f389 ready!

Name Link
🔨 Latest commit a59e207
🔍 Latest deploy log https://app.netlify.com/sites/benchmarking-polite-crostata-92f389/deploys/6735ffaef71856000892288e
😎 Deploy Preview https://deploy-preview-121--benchmarking-polite-crostata-92f389.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JavierCladellas JavierCladellas merged commit aedaa37 into master Nov 15, 2024
15 checks passed
@JavierCladellas JavierCladellas deleted the 111-new-file-for-plots branch November 15, 2024 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement A new feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

new file for plots

3 participants