Skip to content

aten.sin, aten.cos, aten.neg #3706

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

Closed
wants to merge 1 commit into from
Closed

Conversation

huiyujie
Copy link

Summary:
Implement 3 unary ops:

- func: sin(Tensor self) -> Tensor
- func: cos(Tensor self) -> Tensor
- func: neg(Tensor self) -> Tensor

Reviewed By: derekxu, jorgep31415

Differential Revision: D57646317

Copy link

pytorch-bot bot commented May 22, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/3706

Note: Links to docs will display an error until the docs builds have been completed.

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 22, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D57646317

huiyujie pushed a commit to huiyujie/executorch that referenced this pull request May 23, 2024
Summary:

Implement 3 unary ops:
```
- func: sin(Tensor self) -> Tensor
- func: cos(Tensor self) -> Tensor
- func: neg(Tensor self) -> Tensor
```

Relax atol and rtol to make it pass some tests. Need customized configuration of atol and rtol in the future

Reviewed By: derekxu, jorgep31415

Differential Revision: D57646317
huiyujie pushed a commit to huiyujie/executorch that referenced this pull request May 23, 2024
Summary:

Implement 3 unary ops:
```
- func: sin(Tensor self) -> Tensor
- func: cos(Tensor self) -> Tensor
- func: neg(Tensor self) -> Tensor
```

Relax atol and rtol to make it pass some tests. Need customized configuration of atol and rtol in the future

Reviewed By: derekxu, jorgep31415

Differential Revision: D57646317
huiyujie pushed a commit to huiyujie/executorch that referenced this pull request May 23, 2024
Summary:

Implement 3 unary ops:
```
- func: sin(Tensor self) -> Tensor
- func: cos(Tensor self) -> Tensor
- func: neg(Tensor self) -> Tensor
```

Relax atol and rtol to make it pass some tests. Need customized configuration of atol and rtol in the future

Reviewed By: derekxu, jorgep31415

Differential Revision: D57646317
@huiyujie huiyujie force-pushed the export-D57646317 branch from 0876706 to 3dd061b Compare May 23, 2024 18:30
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D57646317

huiyujie pushed a commit to huiyujie/executorch that referenced this pull request May 23, 2024
Summary:

Implement 3 unary ops:
```
- func: sin(Tensor self) -> Tensor
- func: cos(Tensor self) -> Tensor
- func: neg(Tensor self) -> Tensor
```

Relax atol and rtol to make it pass some tests. Need customized configuration of atol and rtol in the future

bypass-github-export-checks
bypass-github-pytorch-ci-checks
bypass-github-executorch-ci-checks

Reviewed By: derekxu, jorgep31415

Differential Revision: D57646317
@huiyujie huiyujie force-pushed the export-D57646317 branch from 3dd061b to 7e02832 Compare May 23, 2024 21:37
Summary:

Implement 3 unary ops:
```
- func: sin(Tensor self) -> Tensor
- func: cos(Tensor self) -> Tensor
- func: neg(Tensor self) -> Tensor
```

Relax atol and rtol to make it pass some tests. Need customized configuration of atol and rtol in the future

bypass-github-export-checks
bypass-github-pytorch-ci-checks
bypass-github-executorch-ci-checks

Reviewed By: derekxu, jorgep31415

Differential Revision: D57646317
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D57646317

@huiyujie huiyujie force-pushed the export-D57646317 branch from 7e02832 to 62645f5 Compare May 23, 2024 21:37
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D57646317

huiyujie pushed a commit to huiyujie/executorch that referenced this pull request May 23, 2024
Summary:

Implement 3 unary ops:
```
- func: sin(Tensor self) -> Tensor
- func: cos(Tensor self) -> Tensor
- func: neg(Tensor self) -> Tensor
```

Relax atol and rtol to make it pass some tests. Need customized configuration of atol and rtol in the future

bypass-github-export-checks
bypass-github-pytorch-ci-checks
bypass-github-executorch-ci-checks

Reviewed By: derekxu, jorgep31415

Differential Revision: D57646317
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 2194486.

kirklandsign pushed a commit to kirklandsign/executorch that referenced this pull request May 24, 2024
Summary:
Pull Request resolved: pytorch#3706

Implement 3 unary ops:
```
- func: sin(Tensor self) -> Tensor
- func: cos(Tensor self) -> Tensor
- func: neg(Tensor self) -> Tensor
```

Relax atol and rtol to make it pass some tests. Need customized configuration of atol and rtol in the future

bypass-github-export-checks
bypass-github-pytorch-ci-checks
bypass-github-executorch-ci-checks

Reviewed By: derekxu, jorgep31415

Differential Revision: D57646317

fbshipit-source-id: 3ca45439021684754f10177d62260053dabeb001
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants