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

Update ResNet50 example to work with TensorFlow 2.x #2537

Merged
merged 6 commits into from Jan 19, 2021
Merged

Update ResNet50 example to work with TensorFlow 2.x #2537

merged 6 commits into from Jan 19, 2021

Conversation

JanuszL
Copy link
Contributor

@JanuszL JanuszL commented Dec 10, 2020 •
edited

Signed-off-by: Janusz Lisiecki jlisiecki@nvidia.com

Why we need this PR?

Pick one, remove the rest

  • It updates ResNet50 example to work with TensorFlow 2.x

What happened in this PR?

Fill relevant points, put NA otherwise. Replace anything inside []

  • What solution was applied:
    updates ResNet50 example to work with TensorFlow 2.x
  • Affected modules and functionalities:
    ResNet50 TensorFlow example
  • Key points relevant for the review:
    NA
  • Validation and testing:
    CI
  • Documentation (including examples):
    NA

JIRA TASK: [DALI-1779]

@JanuszL
Copy link
Contributor Author

JanuszL commented Dec 10, 2020

!build

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1884076]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1884076]: BUILD PASSED

@awolant awolant changed the title Update ResNet50 example to work with TensorFlow 2.x [WIP] Update ResNet50 example to work with TensorFlow 2.x Dec 11, 2020
@JanuszL JanuszL force-pushed the update_TF_rn50_example branch 2 times, most recently from 13cb9dc to 096d803 Compare December 11, 2020 12:32
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1893396]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1893468]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1893581]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1893581]: BUILD PASSED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1895104]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1895104]: BUILD PASSED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1900161]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1900162]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1900162]: BUILD PASSED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1900615]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1900615]: BUILD PASSED

@JanuszL JanuszL force-pushed the update_TF_rn50_example branch 2 times, most recently from 11ab98d to c3ef759 Compare December 15, 2020 11:42
@JanuszL
Copy link
Contributor Author

JanuszL commented Dec 15, 2020

!build

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1909148]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1909156]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1909156]: BUILD PASSED

@JanuszL JanuszL changed the title [WIP] Update ResNet50 example to work with TensorFlow 2.x Update ResNet50 example to work with TensorFlow 2.x Dec 15, 2020
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1968316]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1968317]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1968317]: BUILD PASSED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1969713]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1969713]: BUILD PASSED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1974481]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1974481]: BUILD PASSED

@JanuszL
Update ResNet50 example to work with TF 2.x
920b723 
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@JanuszL
Review fixes
9de0a63 
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@JanuszL
More review fixes
c898db3 
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@JanuszL
Bugfixes
c50b834 
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@JanuszL
Adjust params
72cf29c 
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1977565]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1977565]: BUILD PASSED

@JanuszL
More fixes
b7b26d8 
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1980720]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1980720]: BUILD PASSED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1991305]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1991306]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1991305]: BUILD PASSED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1991306]: BUILD FAILED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1991306]: BUILD PASSED

1 similar comment
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1991306]: BUILD PASSED

@JanuszL JanuszL merged commit 1b0053d into NVIDIA:master Jan 19, 2021
@JanuszL JanuszL deleted the update_TF_rn50_example branch January 19, 2021 09:45
TheTimmy pushed a commit to TheTimmy/DALI that referenced this pull request Jan 20, 2021
@JanuszL @TheTimmy
Update ResNet50 example to work with TensorFlow 2.x (NVIDIA#2537)
3fa6152 
* Update ResNet50 example to work with TF 2.x

Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jantonguirao jantonguirao jantonguirao approved these changes

@banasraf banasraf banasraf approved these changes

Assignees

@jantonguirao jantonguirao

@banasraf banasraf

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@JanuszL @dali-automaton @jantonguirao @banasraf