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

Refactor core of package to remat package #1

Merged
merged 22 commits into from Sep 19, 2019
Merged

Refactor core of package to remat package #1

merged 22 commits into from Sep 19, 2019

Conversation

@parasj
Copy link
Owner

parasj commented Sep 18, 2019

In this PR, I am cleaning up code and beginning an incremental refactor into a remat package. This PR does not migrate over integration code (i.e. TensorFlow 2.0 integration) nor evaluation code.

@parasj parasj requested a review from ajayjain Sep 18, 2019
@parasj parasj self-assigned this Sep 18, 2019
remat/core/utils/graph_utils.py Outdated Show resolved Hide resolved
@parasj parasj changed the title Begin incremental refactoring into remat package Refactor core of package to remat package Sep 19, 2019
@parasj parasj force-pushed the refactor/subdir branch from 740dd66 to 0ff9d66 Sep 19, 2019
@parasj

This comment has been minimized.

Copy link
Owner Author

parasj commented Sep 19, 2019

@ajayjain I'm going to just go ahead and squash this with no review since this is becoming a monster PR

@parasj parasj merged commit d0da9ca into master Sep 19, 2019
@parasj parasj deleted the refactor/subdir branch Sep 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.