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

Fix issues to support latest version of TensorFlow. #2216

Merged
merged 26 commits into from
Aug 18, 2023
Merged

Fix issues to support latest version of TensorFlow. #2216

merged 26 commits into from
Aug 18, 2023

Conversation

fatcat-z
Copy link
Collaborator

@fatcat-z fatcat-z commented Aug 7, 2023 •
edited
Loading

  1. To support tensorflow 2.12 and 2.13 version, fix relative issues.
  2. Update CI pipeline to include new version in it.

@fatcat-z
Try latest tf.
51a7dc7 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Change _captures to captures.
9df3623 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Replace _captures function.
5b27ea7 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Restore the _captures.
339faa2 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@yan12125 yan12125 mentioned this pull request Aug 10, 2023
Closed
@fatcat-z
Update code to avoid keras version error.
1353884 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Add new version and opset.
fbc9275 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Remove __version__.
577e752 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
RightShift
552d97f 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@jcwchen jcwchen mentioned this pull request Aug 15, 2023
Merged
@fatcat-z
Skip one test case.
4fc2231 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Fix pylint issue.
7fc761d 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Removed unnecessary import.
b3ada56 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Evaluate tests.
9f86a83 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Correct the version supports tf 1.15.
df32556 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Check keras versions.
c68deaa 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z fatcat-z changed the title [DO NOT MERGE]Try latest tf. Fix issues to support latest version of TensorFlow. Aug 17, 2023
@fatcat-z
Fix the max version of onnxruntime for tf 1.15.
4385b96 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Refactor the tests in CI pipeline.
4c031c8 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Re-org CI tests.
b0b95df 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Update .gitignore file.
4326dee 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z fatcat-z marked this pull request as ready for review August 18, 2023 02:44
@fatcat-z fatcat-z requested a review from xiaowuhu August 18, 2023 02:45
@fatcat-z
Update pretrained_model_test CI.
da1a80b 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Correct the way of creating unit test jobs.
4ede2a5 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Remove one duplicated test set.
fb313fc 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Update variable in unit test template.
18e2842 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Correct the variable.
a1e34fb 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Update environment variable.
e623285 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z
Add some temp parameters.
7fb8547 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z fatcat-z enabled auto-merge (squash) August 18, 2023 09:12
@fatcat-z fatcat-z disabled auto-merge August 18, 2023 09:26
@fatcat-z
Adjust tests in unit_test.
f50095d 
Signed-off-by: Jay Zhang <jiz@microsoft.com>
@fatcat-z fatcat-z enabled auto-merge (squash) August 18, 2023 09:35
@fatcat-z fatcat-z merged commit a0bd3e5 into onnx:main Aug 18, 2023
51 checks passed
@yan12125 yan12125 mentioned this pull request Aug 19, 2023
Open
@fatcat-z fatcat-z deleted the upgrade_ci_tf213 branch November 21, 2023 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xiaowuhu xiaowuhu xiaowuhu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fatcat-z @xiaowuhu