Skip to content

[jit] Make PythonPrint a class#26787

Closed
zdevito wants to merge 20 commits intogh/zdevito/117/basefrom
gh/zdevito/117/head
Closed

[jit] Make PythonPrint a class#26787
zdevito wants to merge 20 commits intogh/zdevito/117/basefrom
gh/zdevito/117/head

Conversation

@zdevito
Copy link
Contributor

@zdevito zdevito commented Sep 25, 2019

Stack from ghstack:

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

Differential Revision: D17566440

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
Copy link
Member

@suo suo left a comment

Choose a reason for hiding this comment

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

sure

[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
[jit] Make PythonPrint a class

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

gh-metadata: pytorch pytorch 26787 gh/zdevito/117/head
@facebook-github-bot
Copy link
Contributor

@zdevito merged this pull request in 3de3474.

@facebook-github-bot facebook-github-bot deleted the gh/zdevito/117/head branch October 28, 2019 22:23
thiagocrepaldi pushed a commit to thiagocrepaldi/pytorch that referenced this pull request Feb 4, 2020
Summary:
Pull Request resolved: pytorch#26787

A follow up PR will remove the need to issue import statements,
or write classes in order since they are no longer needed.
 This change allows the same PythonPrint class
to be used for an entire file which will be needed in that patch.

Test Plan: Imported from OSS

Differential Revision: D17566440

Pulled By: zdevito

fbshipit-source-id: 1ee896da0cdfe6a003298e1d4b0238403b9ed6dd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: pybind Related to our Python bindings / interactions with other Python libraries oncall: jit Add this issue/PR to JIT oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants