Update API reference link & notes about `rebalance_frequency`

assaf / vanity

Experiment Driven Development for Ruby

http://vanity.labnotes.org

MIT License

1.55k stars 269 forks source link

Update API reference link & notes about `rebalance_frequency` #310

Closed sdhull closed 7 years ago

sdhull commented 7 years ago

So after digging through this code for a few days, I realized that the "exploitation" and "exploration" mentioned in the docs won't actually be done unless you specify a rebalance_frequency. Seemed like that should be mentioned.

Additionally, the API Reference link is currently broken. This was (apparently) fixed in an earlier commit but docs were never updated.

kweydt commented 7 years ago

👍

phillbaker commented 7 years ago

Thanks! Can you also do a PR to the master branch for the source changes to ab_testing.html ?

sdhull commented 7 years ago

Sure thing.

On Wed, Nov 16, 2016 at 3:11 PM, Phillip Baker notifications@github.com wrote:

Merged #310 https://github.com/assaf/vanity/pull/310.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/assaf/vanity/pull/310#event-861957329, or mute the thread https://github.com/notifications/unsubscribe-auth/AADDd8SQnCw20Lmz-Chuv2Y_beNpP0uhks5q-42WgaJpZM4K0e72 .