-
Notifications
You must be signed in to change notification settings - Fork 373
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
Update OSS black linter version to 24.4.0 #1001
Conversation
This pull request was exported from Phabricator. Differential Revision: D56340613 |
Summary: There are some [linter errors](https://github.com/facebookincubator/AITemplate/actions/runs/8680791999/job/23802124952) in OSS trunk caused by D54447738 updating black version to 2024 in fbcode. Here we match the black version in OSS. Differential Revision: D56340613
435d265
to
be21733
Compare
This pull request was exported from Phabricator. Differential Revision: D56340613 |
@@ -87,6 +87,11 @@ def _find_fusion_root(tensor: Tensor, fusion_patterns: List[Any]) -> int: | |||
dst_op_tensor = dst_op._attrs["outputs"] | |||
if len(dst_op_tensor) != 1: | |||
break | |||
if dst_op_tensor[0]._attrs["is_output"]: |
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.
are these changes expected?
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.
No, this was exported from other diff. Let me disconnect this one and re-export.
Summary: There are some [linter errors](https://github.com/facebookincubator/AITemplate/actions/runs/8680791999/job/23802124952) in OSS trunk caused by D54447738 updating black version to 2024 in fbcode. Here we match the black version in OSS. Reviewed By: hl475 Differential Revision: D56340613
be21733
to
cf6ac86
Compare
This pull request was exported from Phabricator. Differential Revision: D56340613 |
Summary: There are some [linter errors](https://github.com/facebookincubator/AITemplate/actions/runs/8680791999/job/23802124952) in OSS trunk caused by D54447738 updating black version to 2024 in fbcode. Here we match the black version in OSS. Reviewed By: hl475 Differential Revision: D56340613
cf6ac86
to
3bbd749
Compare
This pull request was exported from Phabricator. Differential Revision: D56340613 |
Summary: There are some [linter errors](https://github.com/facebookincubator/AITemplate/actions/runs/8680791999/job/23802124952) in OSS trunk caused by D54447738 updating black version to 2024 in fbcode. Here we match the black version in OSS. Reviewed By: hl475 Differential Revision: D56340613
3bbd749
to
696e19e
Compare
This pull request was exported from Phabricator. Differential Revision: D56340613 |
This pull request has been merged in 0973303. |
Summary: There are some linter errors in OSS trunk caused by D54447738 updating black version to 2024 in fbcode. Here we match the black version in OSS.
Differential Revision: D56340613