Introduce enable/disable experiment cohorting#615
Conversation
|
Thanks for the Pull Request @robin-phung ✨ I wanna do some testing just to make sure it's all good before merging. The |
|
@andrehjr Let me know what your thoughts are on refactoring as I'm also interested (and may be able to assist/collab) in a refactor of the Helper#ab_test ~> Trial#choose!. |
andrehjr
left a comment
There was a problem hiding this comment.
Just a few changes and we should be good to merge. 🚀 Sorry for taking so long.
Being able to 'soft close' an experiment does look useful to give more control over experiments.
Sure! Help is always welcome. I've started the other day a few spikes starting with the ab_test method in #623 in order to start preparing things. |
|
Updated PR with the following changes:
|
|
@robin-phung can you rebase these changes? 18c0514 is already on master 648576a And that's it! 🚀 |
f06b16b to
f9fc878
Compare
|
My rebase went out of whack, it should be resolved now. |
What problem does this solve?
Disabling Cohorting: Allow existing registered participants time to convert while no longer accepting new participants into the experiment. New participants will be given the control will be given the control and will not be recorded as being apart of the experiment.
Why is this useful?
How does this solve it?