Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[jit] infer RRef type as container type #33369

Closed
wants to merge 6 commits into from

Conversation

wanchaol
Copy link
Contributor

@wanchaol wanchaol commented Feb 15, 2020

Stack from ghstack:

This PR add RRef type infer rule when we try to infer a type from a
pyobject, this allow script module attributes could contain a rref,
(i.e. List[RRefs] as a module attribute)

Differential Revision: D19918320

This PR add RRef type infer rule when we try to infer a type from a
pyobject, this allow script module attributes could contain a rref,
(i.e. List[RRefs] as a module attribute)

[ghstack-poisoned]
@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Feb 15, 2020
This PR add RRef type infer rule when we try to infer a type from a
pyobject, this allow script module attributes could contain a rref,
(i.e. List[RRefs] as a module attribute)

[ghstack-poisoned]
@dr-ci
Copy link

dr-ci bot commented Feb 15, 2020

💊 CircleCI build failures summary and remediations

As of commit b8cfd65:

Commit b8cfd65 was recently pushed. Waiting for builds...


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.

This comment has been revised 20 times.

This PR add RRef type infer rule when we try to infer a type from a
pyobject, this allow script module attributes could contain a rref,
(i.e. List[RRefs] as a module attribute)

Differential Revision: [D19918320](https://our.internmc.facebook.com/intern/diff/D19918320)

[ghstack-poisoned]
This PR add RRef type infer rule when we try to infer a type from a
pyobject, this allow script module attributes could contain a rref,
(i.e. List[RRefs] as a module attribute)

Differential Revision: [D19918320](https://our.internmc.facebook.com/intern/diff/D19918320)

[ghstack-poisoned]
This PR add RRef type infer rule when we try to infer a type from a
pyobject, this allow script module attributes could contain a rref,
(i.e. List[RRefs] as a module attribute)

Differential Revision: [D19918320](https://our.internmc.facebook.com/intern/diff/D19918320)

[ghstack-poisoned]
This PR add RRef type infer rule when we try to infer a type from a
pyobject, this allow script module attributes could contain a rref,
(i.e. List[RRefs] as a module attribute)

Differential Revision: [D19918320](https://our.internmc.facebook.com/intern/diff/D19918320)

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

@wanchaol merged this pull request in 2448c97.

hczhu pushed a commit that referenced this pull request Feb 28, 2020
Summary:
Pull Request resolved: #33369

This PR add RRef type infer rule when we try to infer a type from a
pyobject, this allow script module attributes could contain a rref,
(i.e. List[RRefs] as a module attribute)

Test Plan: Imported from OSS

Differential Revision: D19918320

Pulled By: wanchaol

fbshipit-source-id: e5fd99c0ba5693b22ed48f0c0550b5e1dac89990
facebook-github-bot pushed a commit that referenced this pull request Mar 1, 2020
Summary:
Pull Request resolved: #33992

resubmit of #33369 with tweaks on when the rref type being created to ensure ivalue->type() hold the correct RRef type inside of inner element type.

Test Plan: Imported from OSS

Differential Revision: D20175043

Pulled By: wanchaol

fbshipit-source-id: a08b178e989c995632374e6c868d23c5a85526ae
@facebook-github-bot facebook-github-bot deleted the gh/wanchaol/82/head branch March 1, 2020 15:18
ttumiel pushed a commit to ttumiel/pytorch that referenced this pull request Mar 4, 2020
Summary:
Pull Request resolved: pytorch#33369

This PR add RRef type infer rule when we try to infer a type from a
pyobject, this allow script module attributes could contain a rref,
(i.e. List[RRefs] as a module attribute)

Test Plan: Imported from OSS

Differential Revision: D19918320

Pulled By: wanchaol

fbshipit-source-id: e5fd99c0ba5693b22ed48f0c0550b5e1dac89990
ttumiel pushed a commit to ttumiel/pytorch that referenced this pull request Mar 4, 2020
Summary:
Pull Request resolved: pytorch#33992

resubmit of pytorch#33369 with tweaks on when the rref type being created to ensure ivalue->type() hold the correct RRef type inside of inner element type.

Test Plan: Imported from OSS

Differential Revision: D20175043

Pulled By: wanchaol

fbshipit-source-id: a08b178e989c995632374e6c868d23c5a85526ae
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged oncall: jit Add this issue/PR to JIT oncall triage queue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants