search

CarnegieLearningWeb / UpGrade

Framework for adding A/B testing to education applications
https://www.upgradeplatform.org/
BSD 3-Clause "New" or "Revised" License
26 stars 13 forks source link

Patch code for XPRIZE and Demonstration phase #360

Closed amurphy-cl closed 2 years ago

amurphy-cl commented 2 years ago

We need to decide what to do with the current xprize patch code and how it should be maintained if we're advanced to the demonstration phase (probably starting in Sept 22). Do we need to create a more permanent solution to the issue of experiment point/id conflicts?

danoswaltCL commented 2 years ago

Per @SritterCL We need permanent solution, but may not need to be a summer priority

danoswaltCL commented 2 years ago

We should merge this patch into the dev branch for now since it may not be fixed for next release.

mswartzCL commented 2 years ago

Moved this to Summer Release 2 to limit the number of large changes in Summer Release 1.

danoswaltCL commented 2 years ago

maybe i missed something on this in the meeting, what is the intent of this issue? If it's to merge in the patch code, that was already done by jenn. If it's to put in a more permanent solution, I was thinking that was not part of summer scope actually.

@amurphy-cl @SritterCL @jreddig

amurphy-cl commented 2 years ago

@danoswaltCL I think the decision to merge the patch code was made while I was out, so this issue can be closed. For tracking a more permanent solution to this problem, we can create a new issue.