-
Notifications
You must be signed in to change notification settings - Fork 346
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
fix: Legacy CI pip
installation
#2239
Conversation
53b746a
to
d79ed9b
Compare
d79ed9b
to
3d404b9
Compare
9333bbc
to
6b6593c
Compare
toolchains special toml for legacy |
6b6593c
to
28ba56f
Compare
e03e94d
to
01bb7fb
Compare
01bb7fb
to
158ada3
Compare
1507b13
to
fe16044
Compare
@@ -93,13 +100,14 @@ def _get_target_ir(module_type: _ModuleType, ir: str) -> _IRType: | |||
else: | |||
if ir == "default": | |||
# Options are listed in order of preference | |||
if module_is_fxable: | |||
if DYNAMO_ENABLED and module_is_fxable: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update default path to be TorchScript if the install is with a legacy Torch version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great
config.yaml
to support legacy CI testing, including upgraded.toml
andWORKSPACE
files built specifically for the legacy versioncompile()
to be TorchScript in the legacy install