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

Remove onnx_backend_test.py from top level cmake #2256

Merged
merged 5 commits into from
Oct 3, 2023

Conversation

attila-dusnoki-htec
Copy link
Contributor

Currently when anything is changed in onnx_backend_test.py, a brand new build will be triggered.
This test is added correctly here to cmake.
This seems like an old artifact, or maybe i missed something. If so, please close this PR.

@codecov
Copy link

codecov bot commented Sep 27, 2023

Codecov Report

Merging #2256 (2969a8b) into develop (4188c38) will increase coverage by 0.00%.
Report is 5 commits behind head on develop.
The diff coverage is 100.00%.

❗ Current head 2969a8b differs from pull request most recent head f558d21. Consider uploading reports for the commit f558d21 to get more accurate results

@@           Coverage Diff            @@
##           develop    #2256   +/-   ##
========================================
  Coverage    91.49%   91.50%           
========================================
  Files          430      430           
  Lines        16129    16133    +4     
========================================
+ Hits         14758    14762    +4     
  Misses        1371     1371           
Files Coverage Δ
src/onnx/parse_constant_of_shape.cpp 92.85% <100.00%> (+1.19%) ⬆️

@umangyadav umangyadav requested a review from pfultz2 September 27, 2023 21:08
@causten causten merged commit bd425d0 into ROCm:develop Oct 3, 2023
14 of 15 checks passed
@causten causten deleted the cmake_python_backend_test branch October 3, 2023 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants