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.

Sign up for GitHub

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

Jump to bottom

Change ADVI to use updates.py #1784

Closed
twiecki opened this issue Feb 13, 2017 · 1 comment
Closed

Change ADVI to use updates.py #1784

twiecki opened this issue Feb 13, 2017 · 1 comment
Labels
beginner friendly

Comments

@twiecki
Copy link
Member

twiecki commented Feb 13, 2017

ADVI currently still has its own optimizer implemented, we can just change it to use the new one in updates.py.
@ferrine
Copy link
Member

ferrine commented Feb 13, 2017

Adam seems to be the default choice
BTW I'm planning to change advi after #1694 and add deprecation warning.

@bhargavvader bhargavvader mentioned this issue Feb 25, 2017
Closed
@twiecki twiecki closed this as completed Mar 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
beginner friendly
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@twiecki @ferrine