Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Separate plots-config from benchmark #160

Closed
JavierCladellas opened this issue Nov 26, 2024 · 0 comments · Fixed by #204
Closed

Separate plots-config from benchmark #160

JavierCladellas opened this issue Nov 26, 2024 · 0 comments · Fixed by #204
Assignees
@JavierCladellas
Labels
enhancement A new feature or request

Comments

@JavierCladellas
Copy link
Collaborator

JavierCladellas commented Nov 26, 2024 •
edited
Loading

The plots-config option should appear for render-benchmarks rather than for execute-benchmark. Move all code related to plots to the other script.
Handle CI conflicts that this might cause.

Maybe add options to bypass certain steps of the benchmarking pipeline to update reports.
@JavierCladellas JavierCladellas self-assigned this Nov 26, 2024
@JavierCladellas JavierCladellas added the enhancement A new feature or request label Nov 26, 2024
JavierCladellas added a commit that referenced this issue Dec 18, 2024
@JavierCladellas
split parser from main #160
806880c
JavierCladellas added a commit that referenced this issue Dec 18, 2024
@JavierCladellas
add replacePlotConfig method #160
80542b0
JavierCladellas added a commit that referenced this issue Dec 18, 2024
@JavierCladellas
add validation and save patches option #160
0c7f355
JavierCladellas added a commit that referenced this issue Dec 18, 2024
@JavierCladellas
add repository method to patch plots config #160
735c6d5
@JavierCladellas JavierCladellas mentioned this issue Dec 18, 2024
Merged
4 tasks
@JavierCladellas JavierCladellas linked a pull request Dec 18, 2024 that will close this issue
resolves : 160 separate plots config from benchmark #204
Merged
4 tasks
JavierCladellas added a commit that referenced this issue Dec 18, 2024
@JavierCladellas
config_file -> config-file #160
49af158
JavierCladellas added a commit that referenced this issue Dec 19, 2024
@JavierCladellas
use time start instead of time end #160
aacea53
JavierCladellas added a commit that referenced this issue Dec 19, 2024
@JavierCladellas
bug fixes #160
be30cf5
JavierCladellas added a commit that referenced this issue Dec 19, 2024
@JavierCladellas
add --website flag to automatically compile antora docs and start htt…
93cb727 
…p server #160
JavierCladellas added a commit that referenced this issue Dec 19, 2024
@JavierCladellas
exception #160
fc7c65e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JavierCladellas JavierCladellas

Labels
enhancement A new feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

resolves : 160 separate plots config from benchmark feelpp/benchmarking
1 participant
@JavierCladellas