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

Quit if code changed between two trials execution #429

Merged
merged 1 commit into from
Jul 16, 2020

Conversation

bouthilx
Copy link
Member

Why:

We test the code version at beginning of experiment but it could change
during an execution, leading to trials ran with different code versions.

How:

Validate before the execution of the trial that code version did not
change. If so, raise BranchingEvent().

Note:

There may be race conditions between the validation and the execution of the trial.
The only solution for this would be to test the code version at
beginning of trial execution but that would require reimplementation for
all languages of users' scripts...

Why:

We test the code version at beginning of experiment but it could change
during an execution, leading to trials ran with different code versions.

How:

Validate before the execution of the trial that code version did not
change. If so, raise BranchingEvent().

Note:

There may be race conditions between the validation and the execution of the trial.
The only solution for this would be to test the code version at
beginning of trial execution but that would require reimplementation for
all languages of users' scripts...
@bouthilx bouthilx added the bug Indicates an unexpected problem or unintended behavior label Jul 16, 2020
@bouthilx bouthilx added this to the v0.1.9 milestone Jul 16, 2020
@bouthilx bouthilx requested a review from Thomsch July 16, 2020 18:31
@bouthilx
Copy link
Member Author

@bouthilx bouthilx merged commit 4bdfd0c into Epistimio:develop Jul 16, 2020
This was referenced Nov 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants