Skip to content

Conversation

@842974287
Copy link
Contributor

Summary: allow int type input of batch_splits

Test Plan:

buck test caffe2/caffe2/python/operator_test:torch_integration_test -- test_box_with_nms_limits

Reviewed By: jackm321

Differential Revision: D24629522

Summary: allow int type input of `batch_splits`

Test Plan:
```
buck test caffe2/caffe2/python/operator_test:torch_integration_test -- test_box_with_nms_limits
```

Reviewed By: jackm321

Differential Revision: D24629522

fbshipit-source-id: c0e1e4c7516686f9f504985a208eda35236b1c4e
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D24629522

@dr-ci
Copy link

dr-ci bot commented Nov 6, 2020

💊 CI failures summary and remediations

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


💚 💚 Looks good so far! There are no failures yet. 💚 💚


1 failure confirmed as flaky and can be ignored:

  • pytorch_linux_xenial_py3_6_gcc5_4_build

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 2 times.

@codecov
Copy link

codecov bot commented Nov 6, 2020

Codecov Report

Merging #47504 (0848881) into master (1aeefcd) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #47504   +/-   ##
=======================================
  Coverage   80.81%   80.81%           
=======================================
  Files        1806     1806           
  Lines      189822   189822           
=======================================
+ Hits       153407   153408    +1     
+ Misses      36415    36414    -1     

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in c19eb4a.

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.

2 participants