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

Case2272044573 merge for 1.5 1 #298

Merged
merged 9 commits into from
Jun 20, 2022
Merged

Case2272044573 merge for 1.5 1 #298

merged 9 commits into from
Jun 20, 2022

Conversation

NikhilRaverkar
Copy link
Contributor

@NikhilRaverkar NikhilRaverkar commented Jun 16, 2022

Issue #, if available:

  • Commiting the changes for Case2272044573 to 1.5 branch
  • Integ tests for 1.3 passed
  • this change is also pushing 2 HP validations we added for 1.3, making it consistent with 1.3 image.
    Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Nikhil Raverkar and others added 9 commits June 15, 2022 14:40
* removing duplicate files from data_path while creating symlink

* not creating duplicate symlinks to resolve FileExistsError

* not creating duplicate symlinks to resolve FileExistsError

* not creating duplicate symlinks to resolve FileExistsError

* not creating duplicate symlinks to resolve FileExistsError

* not creating duplicate symlinks to resolve FileExistsError

* not creating duplicate symlinks to resolve FileExistsError

* not creating duplicate symlinks to resolve FileExistsError

* not creating duplicate symlinks to resolve FileExistsError

* not creating duplicate symlinks to resolve FileExistsError

* not creating duplicate symlinks to resolve FileExistsError

* Add warnings when validation files are suspected to be identical with training files (#273)

* Add warnings when validation files are suspected to be identical with training files

* Resolving comments

* Add warnings when validation files are suspected to be identical with training files (#273)

* Add warnings when validation files are suspected to be identical with training files

* Resolving comments

* remove the return statement in check redundancy function

* fix some format issues

* fixing Nvidia key error

* fixing Nvidia key error

* fixing Nvidia key error

* add debug logs

* add default value for validate path

* check if validation path is not set

* resolve comments on logging

Co-authored-by: Nikhil Raverkar <nraverka@amazon.com>
Co-authored-by: Haixin Wang <98612668+haixiw@users.noreply.github.com>
Co-authored-by: haixiw <haixiw@amazon.com>
* Revert "Case2272044573 master merge (#285)"

This reverts commit 81405a6.

* Fix NVIDIA keyring error

* Bump docker runtime from 17 to 19

* Delete numpy-1.21.6 site packages

* Freeze protobuf to 3.20.1

Co-authored-by: Mark Bunday <mabunday@amazon.com>
@NikhilRaverkar NikhilRaverkar changed the title Case2272044573 1.5 1 Case2272044573 merge for 1.5 1 Jun 16, 2022
@NikhilRaverkar NikhilRaverkar marked this pull request as ready for review June 16, 2022 18:25
@NikhilRaverkar NikhilRaverkar merged commit 9556801 into master Jun 20, 2022
@mabunday mabunday deleted the Case2272044573_1.5-1 branch August 4, 2022 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants