Skip to content

Make it possible to disable JIT using env variables#10867

Closed
apaszke wants to merge 3 commits intopytorch:masterfrom
apaszke:jit_disable
Closed

Make it possible to disable JIT using env variables#10867
apaszke wants to merge 3 commits intopytorch:masterfrom
apaszke:jit_disable

Conversation

@apaszke
Copy link
Contributor

@apaszke apaszke commented Aug 24, 2018

script_method = self._get_method(attr)
return functools.wraps(original_method)(script_method)
if _enabled:
class ScriptModule(with_metaclass(ScriptMeta, torch._C.ScriptModule, Module)):

This comment was marked as off-topic.

@zou3519 zou3519 added 0.3 oncall: jit Add this issue/PR to JIT oncall triage queue and removed 0.3 labels Aug 28, 2018
Copy link
Contributor

@zdevito zdevito left a comment

Choose a reason for hiding this comment

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

Looks good. There seem to be related test errors.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

apaszke has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

apaszke has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

PenghuiCheng pushed a commit to PenghuiCheng/pytorch that referenced this pull request Sep 11, 2018
Summary:
zdevito
Pull Request resolved: pytorch#10867

Differential Revision: D9556882

Pulled By: apaszke

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

Labels

oncall: jit Add this issue/PR to JIT oncall triage queue open source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants