Skip to content

Conversation

smessmer
Copy link
Contributor

@smessmer smessmer commented Apr 1, 2019

Stack:
    :black_circle:  #18706 Enable gmock and fix system gtest issue  💛
    :white_circle:  #18547 Improve and test error messages for signature mismatches  💚
    :white_circle:  #18550 Improve compiler error messages of the op registration API  💚
    :white_circle:  #18551 Allow registering an operator schema without a kernel  💚
    :white_circle:  #18577 Register operators by passing arguments to RegisterOperators constructor  💚
    :white_circle:  #18592 [wip] Move ATen-core target one directory up  💛
    :white_circle:  #18593 [wip] Split function schema parser from operator  💛
    :white_circle:  #18594 [wip] Use function schema parser from op registration API  💛
    :white_circle:  #18597 [wip] Rewrite test cases with string based schema  💛
    :white_circle:  #18599 [wip] Use string based schema for exposing caffe2 ops  💛
    :white_circle:  #18600 [wip] Remove FunctionSchema based registration APIs  💛

  • Enable gmock
  • Fix issue where the gtest source files in third_party would include system gtest headers

Differential Revision: D14715302

Differential Revision: D14715302
Differential Version: 77672605
smessmer added 2 commits April 1, 2019 15:27
Differential Revision: D14715302
Differential Version: 77673988
Differential Revision: D14715302
Differential Version: 77725327
Copy link
Contributor

@ezyang ezyang left a comment

Choose a reason for hiding this comment

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

Approved with comments

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in bb8a0d7.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Apr 2, 2019
Summary:
Pull Request resolved: pytorch/pytorch#18706

- Enable gmock
- Fix issue where the gtest source files in third_party would include system gtest headers

Reviewed By: ezyang

Differential Revision: D14715302

fbshipit-source-id: 5335390913e651bda85c69d7ea9b5c1bce58f172
@ezyang ezyang deleted the export-D14715302 branch May 30, 2019 15: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.

3 participants