-
Notifications
You must be signed in to change notification settings - Fork 1.2k
TF-2.16 test modification and handling #4830
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
Merged
Merged
Changes from 21 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
50cf9aa
[DO NOT MERGE] Experimenting TF image_uri configs
shantanutrip 80f7875
Add logic to Tf estimator
shantanutrip 0a2ed3d
Comment previous data
shantanutrip 2a2dfd0
Test with making changes to model.py
shantanutrip e359e21
Add override FW version
shantanutrip c95a787
Make changes to prevent non 2.16 from making the change
shantanutrip dc522e6
Add Print
shantanutrip 5625988
Merge remote-tracking branch 'origin/master' into test-tf-2-16-inf
shantanutrip 3036cce
Use hasattr
shantanutrip 5711a20
Fix print and use getattr
shantanutrip c053583
Print statement
shantanutrip 0529828
Only define override when needed
shantanutrip 8690405
Change to net.export() for TF2.16
shantanutrip 3ad5961
Skip tests failing due to TF-IO
shantanutrip a2b424d
Reformatting
shantanutrip 7a13af7
Reformatting pylint
shantanutrip 4147f69
Change tf_full_vesion fixture
shantanutrip 48c2507
Add Version
shantanutrip d1a3475
Revert unit test changes
shantanutrip 8493136
Preventing re-initialization of Version
shantanutrip 3e55e7d
Merge branch 'master' into test-tf-2-16-inf
shantanutrip 5e6ff38
Revert config JSON changes
shantanutrip 3f11254
Handle in case inf and training have different major.minor
shantanutrip 7c5add1
Introduce return version concept in tests
shantanutrip 3c7135e
Merge branch 'master' into test-tf-2-16-inf
shantanutrip f7cf9d3
Add TF2.16.1 inf to config
shantanutrip d88af35
Revert temp changes
shantanutrip 3751a63
Merge branch 'master' into test-tf-2-16-inf
benieric File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -331,7 +331,8 @@ | |
"2.11": "2.11.1", | ||
"2.12": "2.12.1", | ||
"2.13": "2.13.0", | ||
"2.14": "2.14.1" | ||
"2.14": "2.14.1", | ||
"2.16": "2.16.1" | ||
}, | ||
"versions": { | ||
"1.4.1": { | ||
|
@@ -2188,6 +2189,48 @@ | |
"us-west-2": "763104351884" | ||
}, | ||
"repository": "tensorflow-inference" | ||
}, | ||
"2.16.1": { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same for this, will be reverted and added via auto-commits |
||
"registries": { | ||
"af-south-1": "626614931356", | ||
"ap-east-1": "871362719292", | ||
"ap-northeast-1": "763104351884", | ||
"ap-northeast-2": "763104351884", | ||
"ap-northeast-3": "364406365360", | ||
"ap-south-1": "763104351884", | ||
"ap-south-2": "772153158452", | ||
"ap-southeast-1": "763104351884", | ||
"ap-southeast-2": "763104351884", | ||
"ap-southeast-3": "907027046896", | ||
"ap-southeast-4": "457447274322", | ||
"ca-central-1": "763104351884", | ||
"ca-west-1": "204538143572", | ||
"cn-north-1": "727897471807", | ||
"cn-northwest-1": "727897471807", | ||
"eu-central-1": "763104351884", | ||
"eu-central-2": "380420809688", | ||
"eu-north-1": "763104351884", | ||
"eu-south-1": "692866216735", | ||
"eu-south-2": "503227376785", | ||
"eu-west-1": "763104351884", | ||
"eu-west-2": "763104351884", | ||
"eu-west-3": "763104351884", | ||
"il-central-1": "780543022126", | ||
"me-central-1": "914824155844", | ||
"me-south-1": "217643126080", | ||
"sa-east-1": "763104351884", | ||
"us-east-1": "763104351884", | ||
"us-east-2": "763104351884", | ||
"us-gov-east-1": "446045086412", | ||
"us-gov-west-1": "442386744353", | ||
"us-iso-east-1": "886529160074", | ||
"us-isob-east-1": "094389454867", | ||
"us-isof-east-1": "303241398832", | ||
"us-isof-south-1": "454834333376", | ||
"us-west-1": "763104351884", | ||
"us-west-2": "763104351884" | ||
}, | ||
"repository": "tensorflow-inference" | ||
} | ||
} | ||
}, | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will be reverted and added via automated commits