Skip to content

Move GetExceptionString to Error.h#11501

Closed
smessmer wants to merge 123 commits intomasterfrom
export-D9763424
Closed

Move GetExceptionString to Error.h#11501
smessmer wants to merge 123 commits intomasterfrom
export-D9763424

Conversation

@smessmer
Copy link
Contributor

@smessmer smessmer commented Sep 11, 2018

Stack:
    :black_circle:  #11501 Move GetExceptionString to Error.h  💛
    :white_circle:  #11502 Improve TypeMeta  💛

This doesn't really belong to TypeMeta, moving it to the error handling header

Differential Revision: D9763424

Differential Revision: D9623916
Differential Version: 56705898
Differential Revision: D9623916
Differential Version: 56858422
Differential Revision: D9644700
Differential Version: 56870160
Differential Revision: D9623916
Differential Version: 56900314
Differential Revision: D9644700
Differential Version: 56900331
Differential Revision: D9652088
Differential Version: 56917655
Differential Revision: D9652088
Differential Version: 56918713
Differential Revision: D9652089
Differential Version: 56918714
Differential Revision: D9623916
Differential Version: 56982469
Differential Revision: D9644700
Differential Version: 56982494
Differential Revision: D9652088
Differential Version: 56982465
Differential Revision: D9652089
Differential Version: 56982470
Differential Revision: D9663476
Differential Version: 56982467
Differential Revision: D9623916
Differential Version: 57101548
Differential Revision: D9644700
Differential Version: 57101652
Differential Revision: D9652088
Differential Version: 57101545
Differential Revision: D9652089
Differential Version: 57101550
Differential Revision: D9663476
Differential Version: 57101556
Differential Revision: D9623916
Differential Version: 57135223
Differential Revision: D9644700
Differential Version: 57135224
Differential Revision: D9652088
Differential Version: 57135218
Differential Revision: D9652089
Differential Version: 57135219
Differential Revision: D9663476
Differential Version: 57135226
Differential Revision: D9694327
Differential Version: 57135227
Differential Revision: D9694326
Differential Version: 57135216
Differential Revision: D9694918
Differential Version: 57140103
Differential Revision: D9623916
Differential Version: 57214033
Differential Revision: D9644700
Differential Version: 57214023
Differential Revision: D9652088
Differential Version: 57214005
Differential Revision: D9652089
Differential Version: 57214030
Differential Revision: D9731326
Differential Version: 57851527
Differential Revision: D9775191
Differential Version: 57851522
Differential Revision: D9763423
Differential Version: 57851520
Differential Revision: D9763424
Differential Version: 57851529
Differential Revision: D9728631
Differential Version: 58048804
Differential Revision: D9731326
Differential Version: 58048809
Differential Revision: D9775191
Differential Version: 58048806
Differential Revision: D9763423
Differential Version: 58048802
Differential Revision: D9763424
Differential Version: 58048807
Differential Revision: D9763424
Differential Version: 58217856
@smessmer smessmer changed the base branch from export-D9763423 to master September 18, 2018 20:53
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.

using std::string

Differential Revision: D9763424
Differential Version: 58458886
@smessmer
Copy link
Contributor Author

I don't agree with that style rule, but I reverted the change for now to unblock the PR.

Differential Revision: D9763424
Differential Version: 58502052
Differential Revision: D9763424
Differential Version: 58541570
zdevito pushed a commit to zdevito/ATen that referenced this pull request Sep 22, 2018
Summary:
Pull Request resolved: pytorch/pytorch#11501

This doesn't really belong to TypeMeta, moving it to the error handling header

Reviewed By: ezyang

Differential Revision: D9763424

fbshipit-source-id: 127a8246171ab3a4475f2767d2dc1cc13c486a2e
@soumith soumith deleted the export-D9763424 branch February 21, 2019 23:24
@ezyang ezyang added the merged label Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants