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.

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

Add initial version of benchmarks #50

Merged
merged 34 commits into from
Feb 9, 2024

Conversation

fia0
Copy link

@fia0 fia0 commented Jan 23, 2024

This PR contains a cleaned-up version of the formerly used collection of benchmarks from various theses. Plotting scripts are provided via haura-plots which can be used with poetry. All information is contained in the attached READMEs.

Including the benchmarks in the repository ensures compatibility by running smoke tests in the CI.

@fia0 fia0 force-pushed the feature/add-external-benchmarks branch 2 times, most recently from d357f6d to 55c50c4 Compare January 23, 2024 14:50
@fia0
Copy link
Author

fia0 commented Jan 24, 2024

Some modifications to the CI pipeline were required, they mostly include the building of release artifacts in the dependency job and the benchmarks CI job.

@fia0 fia0 force-pushed the feature/add-external-benchmarks branch from 570d5e2 to 8f28b85 Compare January 25, 2024 11:13
@fia0 fia0 force-pushed the feature/add-external-benchmarks branch from 8f28b85 to 1e20471 Compare January 26, 2024 17:36
@fia0
Copy link
Author

fia0 commented Feb 9, 2024

The PR itself works, the CI fails here as one of the docs dependencies (mdbook-graphviz) has incompatibilities with rust 1.76. Therefore merging will continue.

@fia0 fia0 merged commit 3f8d83c into parcio:main Feb 9, 2024
7 of 8 checks passed
@fia0 fia0 deleted the feature/add-external-benchmarks branch February 9, 2024 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant