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

Update migration strategy #2156

Merged
merged 1 commit into from
Nov 1, 2020

Conversation

seanpmorgan
Copy link
Member

Description

Updates the migration strategy.

cc @dynamicwebpaige @karmel @tomerk @tanzhenyu


**Given the challenges of external SIG coordinating with internal roadmaps, a new
process has been put in place for the core TF and Keras teams to handle migration
and deprecation of Addons components. If you believe there is a strong candidate for
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the real problem here is about expecting a deprecation PR.
I think it is plausible to expect this if we have a "real" migration approach but instead it will be very hard if we have a duplicative process in the "ecosystem".
When you start to duplicate something it is hard to expect that you will care about a deprecation cause probably in the duplication act implicitly you didn't care about checking in the ecosystem.
It is not totally correlated cause there could be real cases when you are constrained to reinvent the wheel but I suppose there is a very strong relationship between the duplication behaviour and the lack of deprecation activities.

Copy link
Contributor

Choose a reason for hiding this comment

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

Just to make a pratical use case:
#2004 (comment)

@seanpmorgan seanpmorgan merged commit c69c44b into tensorflow:master Nov 1, 2020
@seanpmorgan seanpmorgan deleted the update-migration-path branch November 1, 2020 22:21
jrruijli pushed a commit to jrruijli/addons that referenced this pull request Dec 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants