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

[WIP] Experiment serialization and reconstruction #1423

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

ItamarGoldman
Copy link
Contributor

@ItamarGoldman ItamarGoldman commented Mar 14, 2024

Summary

This PR will able the user to save and load experiment object to the result_db. This issue was discussed in #1392

Details and comments

Tasks to complete this PR:

  • make save experiment flag to save experiment config to the result_db

  • Add serialization of both Experiment obj, Analysis obj and figure related obj (for plotter, drawer and figure options)

  • Use the provider to add the correct backend to the experiment object.

  • Make a utility function to load experiment and experiment data simultaneously.

  • Add tests for simple experiment loading and saving

  • Add tests for composite experiments, e.g. flat, nesting etc..

  • This feature was requested by the experimentalists.

PR checklist (delete when all criteria are met)

  • I have read the contributing guide CONTRIBUTING.md.
  • I have added the tests to cover my changes.
  • I have updated the documentation accordingly.
  • I have added a release note file using reno if this change needs to be documented in the release notes.

@ItamarGoldman ItamarGoldman changed the title Experiment serialization and reconstruction [WIP] Experiment serialization and reconstruction Mar 18, 2024
@coruscating coruscating added this to the Release 0.7 milestone Mar 19, 2024
@wshanks wshanks mentioned this pull request Mar 19, 2024
4 tasks
@eliarbel eliarbel self-requested a review March 20, 2024 07:47
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