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

Post-run summary_csv file generation #22

Merged
merged 8 commits into from
Nov 17, 2023
Merged

Post-run summary_csv file generation #22

merged 8 commits into from
Nov 17, 2023

Conversation

TarekAbouChakra
Copy link
Collaborator

Pull Request Summary:

  • Added a new argument to the neps.run function to enable the generation of post-run CSV files.
  • The generated CSV files now include the status of each configuration, along with existing data such as result, metadata, and configuration information. In addition to summarized data about the run in the run_data.csv.
  • Introduced file locking mechanism inspired by the one used in metahyper to ensure smooth operation with multiple workers.
  • Implemented checks on the current CSV file and data for scenarios involving multiple workers, ensuring data integrity.
  • Added a unit test that executes after the metahyper_locking test, validating the functionality introduced in this PR.

@Neeratyoy Neeratyoy merged commit 5d3f851 into master Nov 17, 2023
11 checks passed
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.

2 participants