Closed joncloud closed 8 years ago
Looks good to me. I haven't had a chance to try it out. @davezych do you have any feedback before I merge?
I don't have time to pull it down and try it but it looks good on paper.
As a note for the future, the ExperimentInstance
constructor is getting a bit unwieldy - there's what, 10 parameters there now? We might want to think about somehow cleaning that up sometime.
As a note for the future, the ExperimentInstance constructor is getting a bit unwieldy - there's what, 10 parameters there now? We might want to think about somehow cleaning that up sometime.
Indeed.
Nice work @joncloud!
@Haacked should be good to go.