Skip to content

Conversation

janeyx99
Copy link
Contributor

@janeyx99 janeyx99 commented Nov 3, 2020

Currently, no test reports are uploaded to CI because the paths for the onnx runs are incorrect. This PR attempts to change that.

@dr-ci
Copy link

dr-ci bot commented Nov 3, 2020

💊 CI failures summary and remediations

As of commit d1e9b93 (more details on the Dr. CI page):


  • 1/1 failures introduced in this PR

XLA failure

Job pytorch_xla_linux_bionic_py3_6_clang9_test is failing. Please create an issue with title prefixed by [PT_BREAK] in pytorch/xla and link to to this PR. If you have questions, please reach out to @ailzhang / @dlibenzi / @JackCaoG.


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 10 times.

@janeyx99 janeyx99 force-pushed the onnx-reports branch 2 times, most recently from 25f84b3 to 28c97d9 Compare November 3, 2020 23:11
@codecov
Copy link

codecov bot commented Nov 4, 2020

Codecov Report

Merging #47315 into master will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #47315   +/-   ##
=======================================
  Coverage   60.81%   60.81%           
=======================================
  Files        2749     2749           
  Lines      254093   254093           
=======================================
+ Hits       154526   154528    +2     
+ Misses      99567    99565    -2     

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@janeyx99 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@janeyx99 janeyx99 requested review from malfet and a team November 4, 2020 14:59
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@janeyx99 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Contributor

@malfet malfet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the fix!

@facebook-github-bot
Copy link
Contributor

@janeyx99 merged this pull request in 4189c3c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants