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

Remove deprecated KerasClient #857

Merged
merged 6 commits into from
Mar 1, 2022
Merged

Remove deprecated KerasClient #857

merged 6 commits into from
Mar 1, 2022

Conversation

danieljanes
Copy link
Member

@danieljanes danieljanes commented Sep 25, 2021 •
edited
Loading

Tasks:

  • Remove deprecated KerasClient and start_keras_client
  • Add release notes

@danieljanes danieljanes added this to the 0.18.0 milestone Oct 12, 2021
@mariaboerner1987 mariaboerner1987 added documentation Improvements or additions to documentation enhancement New feature or request labels Dec 17, 2021
@mariaboerner1987
Copy link
Contributor

@danieljanes what is the status of this PR?

@danieljanes danieljanes added the breaking change This change will introduce break change to the user API. label Jan 6, 2022
@danieljanes danieljanes marked this pull request as ready for review March 1, 2022 10:33
@danieljanes
Copy link
Member Author

@mariaboerner1987 we can merge this now that Flower 0.18 is released.

@danieljanes danieljanes enabled auto-merge (squash) March 1, 2022 10:34
@danieljanes danieljanes merged commit 698e592 into main Mar 1, 2022
@danieljanes danieljanes deleted the rm-kerasclient branch March 1, 2022 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tanertopal tanertopal tanertopal approved these changes

Assignees

@danieljanes danieljanes

Labels
breaking change This change will introduce break change to the user API. documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
0.18.0
Development

Successfully merging this pull request may close these issues.
3 participants
@danieljanes @mariaboerner1987 @tanertopal