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

54 epsilon greedy #78

Merged
merged 19 commits into from
Feb 1, 2018
Merged

Conversation

Maximophone
Copy link
Contributor

Added epsilon greedy algorithm to examples. Created a notebook to illustrate the use of an Epsilon Greedy router.
Updated wrappers, cluster managers and various bug fixes.

Copy link
Contributor

@ukclivecox ukclivecox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested the notebook but the example doesn't work for me. Route 2 never seems to get more traffic than route 0.

@ukclivecox ukclivecox merged commit aacd0ad into SeldonIO:master Feb 1, 2018
agrski pushed a commit that referenced this pull request Dec 2, 2022
* initial wiring for http reverse proxy

* server config updates

* lint

* remove enable proxy

* docker compose updates

* Update envoy processing so each route has a unqiue name and is not set to model name.

* Update set state method to take an expected state to transition from

* add missing Docker compose files

* lint

* Add model lock/unlock

* lint

* Add test for unexpected state change

* lint

* review comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants