Fix dynamo.export(aten=True) for condition op #92361
Conversation
[ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/92361
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit b2465b4: This comment was automatically generated by Dr. CI and updates every 15 minutes. |
github-actions
bot
requested review from
albanD,
antoniojkim,
bdhirsh,
Chillee,
ezyang,
jbschlosser,
miladm,
voznesenskym and
wconstab
SherlockNoMad
added
topic: not user facing
SherlockNoMad
requested review from
avikchaudhuri
and removed request for
ezyang,
miladm,
wconstab,
Chillee,
albanD,
bdhirsh,
antoniojkim and
jbschlosser
| @@ -37,7 +37,7 @@ class UnsupportedAliasMutationException(RuntimeError): | |||
|
|
|||
|
|
|||
| def trace_cond(proxy_mode, func_overload, pred, true_fn, false_fn, operands): | |||
| assert isinstance(operands, list), "Cond operands must be a list of tensors" | |||
| assert isinstance(operands, (list, tuple)), "Cond operands must be a list or tuple of tensors" | |||
There was a problem hiding this comment.
operands is passed in as a tuple, instead of a list
|
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Merge failedReason: The following mandatory check(s) failed (Rule Dig deeper by viewing the failures on hud Details for Dev Infra teamRaised by workflow job |
cc mlazos soumith voznesenskym yanboliang penguinwu anijain2305 EikanWang jgong5 Guobing-Chen chunyuan-w XiaobingSuper zhuhaozhe blzheng Xia-Weiwen wenzhe-nrv jiayisunx desertfire [ghstack-poisoned]
|
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Merge failedReason: The following mandatory check(s) failed (Rule Dig deeper by viewing the failures on hud Details for Dev Infra teamRaised by workflow job |
|
@pytorchbot merge -r |
|
@pytorchbot successfully started a rebase job. Check the current status here |
cc mlazos soumith voznesenskym yanboliang penguinwu anijain2305 EikanWang jgong5 Guobing-Chen chunyuan-w XiaobingSuper zhuhaozhe blzheng Xia-Weiwen wenzhe-nrv jiayisunx desertfire [ghstack-poisoned]
|
Successfully rebased |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Stack from ghstack (oldest at bottom):
cc @mlazos @soumith @voznesenskym @yanboliang @penguinwu @anijain2305 @EikanWang @jgong5 @Guobing-Chen @chunyuan-w @XiaobingSuper @zhuhaozhe @blzheng @Xia-Weiwen @wenzhe-nrv @jiayisunx @desertfire