-
Notifications
You must be signed in to change notification settings - Fork 58
release 0.6.0 #33
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
Comments
originally posted by Anton Akhmerov (@anton-akhmerov) at 2018-09-24T18:32:38.668Z on GitLab Because I'm on vacation, I'll abstain on this one. No objections from my side on releasing and version choice. |
originally posted by Jorn Hoofwijk (@Jorn) at 2018-09-25T10:41:35.189Z on GitLab I would like the following to be merged as well: Since this issue is actually much broader than just the balancingLearner + LearnerND, the problem is actually the combination of Async + LearnerND, which I suppose would occur often. |
originally posted by Jorn Hoofwijk (@Jorn) at 2018-09-25T11:30:41.200Z on GitLab Then maybe we can also add gitlab:!113 (gitlab:#108) |
originally posted by Bas Nijholt (@basnijholt) at 2018-10-04T13:36:37.615Z on GitLab All issues and merge requests for 0.6 have been closed and merged. Shall we do it now? @anton-akhmerov, @jorn, and @jbweston? |
(original issue on GitLab)
opened by Bas Nijholt (@basnijholt) at 2018-09-24T16:59:55.276Z
Since 0.5.0 we fixed the following issues:
and merged the following Merge Requests:
New features
learner.tell_pending
which replaceslearner.tell(x, None)
runner.retries
andrunner.raise_if_retries_exceeded
learner.pending_points
andrunner.pending_points
public APIlearner.ask(n, add_data)
->learner.ask(n, tell_pending)
overhead
method to theBlockingRunner
I think we should release 0.6.0 since we introduce some new features. @jbweston and @anton-akhmerov do you have a different opinion?
The text was updated successfully, but these errors were encountered: