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

Refactor stochastic replay #922

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Refactor stochastic replay #922

wants to merge 3 commits into from

Conversation

cabrador
Copy link
Collaborator

@cabrador cabrador commented Dec 18, 2023 •
edited
Loading

Description

This PR refactors aida-stochastic replay using Executor.

TODO:

  • Priming
  • Print statistics after run

Type of change

  • New feature (non-breaking change which adds functionality)
  • Refactoring (changes that do NOT affect functionality)

@cabrador cabrador marked this pull request as ready for review December 18, 2023 17:04
@cabrador cabrador requested review from b-scholz, wsodsong and matejmode and removed request for b-scholz and wsodsong December 18, 2023 17:04
@cabrador cabrador force-pushed the petr/refactor-stochastic-replay branch from b85f611 to 428f1ea Compare December 18, 2023 17:34
@cabrador cabrador force-pushed the petr/refactor-stochastic-replay branch from f2ca7b0 to f9a115f Compare December 18, 2023 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@b-scholz b-scholz Awaiting requested review from b-scholz

@matejmode matejmode Awaiting requested review from matejmode

@wsodsong wsodsong Awaiting requested review from wsodsong

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cabrador