Skip to content
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

[FEAT] sin/cos/tan expression implementation #2112

Merged
merged 1 commit into from
Apr 14, 2024
Merged

Conversation

reswqa
Copy link
Contributor

@reswqa reswqa commented Apr 13, 2024

This closes #2102
This closes #2103

@reswqa reswqa changed the title [FEAT] sin/cos/tan expression implementation [FEAT]: sin/cos/tan expression implementation Apr 13, 2024
@reswqa reswqa changed the title [FEAT]: sin/cos/tan expression implementation [FEAT] sin/cos/tan expression implementation Apr 13, 2024
@reswqa reswqa marked this pull request as ready for review April 13, 2024 16:35
@reswqa
Copy link
Contributor Author

reswqa commented Apr 13, 2024

Not sure how to pass Release Drafter / label (pull_request) as no label are added automatically and I don't have permission to add them either.

@jaychia jaychia added the enhancement New feature or request label Apr 13, 2024
@jaychia
Copy link
Contributor

jaychia commented Apr 13, 2024

I gotchu! :)

Copy link
Member

@samster25 samster25 left a comment

Choose a reason for hiding this comment

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

Wow @reswqa! Extremely clean PR 🥇

T: DaftFloatType,
T::Native: Float,
{
pub fn trigonometry(&self, func: &TrigonometricFunction) -> DaftResult<Self> {
Copy link
Member

Choose a reason for hiding this comment

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

very clean!

@samster25 samster25 merged commit 395c048 into Eventual-Inc:main Apr 14, 2024
28 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[EXPRESSIONS] sin [EXPRESSIONS] tan
3 participants