Skip to content

Conversation

@jerryzh168
Copy link
Contributor

@jerryzh168 jerryzh168 commented Apr 19, 2019

Stack:
    :black_circle:  #19513 [pt1][quant] Add print function for QTensor  💚

Add support for printing a QTensor in python frontend

Differential Revision: D15017168

Differential Revision: D15017168
Differential Version: 80137572
@pytorchbot pytorchbot added module: operators module: printing Issues related to the printing format of tensors labels Apr 19, 2019
Copy link
Contributor

@gchanan gchanan left a comment

Choose a reason for hiding this comment

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

why are we printing both the float values and the int values always? I'm not against it, but would want to hear the reasoning.

Differential Revision: D15017168
Differential Version: 80438721
Differential Revision: D15017168
Differential Version: 80634987
@jerryzh168
Copy link
Contributor Author

@gchanan can you review again

Differential Revision: D15017168
Differential Version: 80637260
@jerryzh168
Copy link
Contributor Author

@gchanan for printing both, I don't have a strong reason for it, mostly just assuming user might want to see both of them since they both represents the values that the QTensor contains.

Differential Revision: D15017168
Differential Version: 80772826
@jerryzh168 jerryzh168 requested a review from gchanan April 26, 2019 01:11
@jerryzh168
Copy link
Contributor Author

@gchanan could you review again?

jerryzh168 added 2 commits May 2, 2019 14:20
Differential Revision: D15017168
Differential Version: 81115670
Differential Revision: D15017168
Differential Version: 81115796
@jerryzh168 jerryzh168 requested a review from gchanan May 2, 2019 21:22
Differential Revision: D15017168
Differential Version: 81115973
Differential Revision: D15017168
Differential Version: 81122150
@jerryzh168 jerryzh168 requested a review from gchanan May 3, 2019 00:47
Copy link
Contributor

@gchanan gchanan left a comment

Choose a reason for hiding this comment

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

lgtm.

jerryzh168 added 2 commits May 3, 2019 13:48
Differential Revision: D15017168
Differential Version: 81180808
Differential Revision: D15017168
Differential Version: 81199014
zdevito pushed a commit to zdevito/ATen that referenced this pull request May 6, 2019
Summary:
Pull Request resolved: pytorch/pytorch#19513

Add support for printing a QTensor in python frontend

Differential Revision: D15017168

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

This pull request has been merged in 17268a9.

@ezyang ezyang deleted the export-D15017168 branch May 30, 2019 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: printing Issues related to the printing format of tensors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants