Skip to content

Experiment loading fixes #51

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

Merged
merged 3 commits into from
Apr 8, 2025
Merged

Experiment loading fixes #51

merged 3 commits into from
Apr 8, 2025

Conversation

ArthurCamara
Copy link
Contributor

This PR adds a couple of simple QoL improvements when loading an experiment.

  1. It assures that the evaluations_cache_path parents exists before creating it
  2. If the output .json file already exists, but a queries .csv file is provided on the creation of the experiment, it will not warn for existing queries.

Verified

This commit was signed with the committer’s verified signature.
j3ko Jeffrey Ko

Verified

This commit was signed with the committer’s verified signature.
j3ko Jeffrey Ko

Verified

This commit was signed with the committer’s verified signature.
j3ko Jeffrey Ko
@ArthurCamara ArthurCamara merged commit 57c494d into master Apr 8, 2025
@ArthurCamara ArthurCamara deleted the experiment-loading-fixes branch April 8, 2025 12:44
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.

None yet

2 participants