Skip to content

[Dynamo] Added Operator Support#131

Merged
yaoyaoding merged 14 commits intohidet-org:mainfrom
AndreSlavescu:operators
Mar 16, 2023
Merged

[Dynamo] Added Operator Support#131
yaoyaoding merged 14 commits intohidet-org:mainfrom
AndreSlavescu:operators

Conversation

@AndreSlavescu
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

@yaoyaoding yaoyaoding left a comment

Choose a reason for hiding this comment

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

Overall LGTM, just a minor comment.

@AndreSlavescu AndreSlavescu changed the title [Dynamo] Sigmoid/Hardsigmoid Operators [Dynamo] Added Operator Support Mar 9, 2023
Copy link
Copy Markdown
Member

@yaoyaoding yaoyaoding left a comment

Choose a reason for hiding this comment

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

Looks good to me. A minor issue is to drop the change for ir/primitives/math.py

@yaoyaoding
Copy link
Copy Markdown
Member

Thanks @AndreSlavescu.

@yaoyaoding yaoyaoding merged commit 3cbaa81 into hidet-org:main Mar 16, 2023
KTong821 pushed a commit to KTong821/hidet that referenced this pull request Apr 24, 2024
The previous implementation is incorrect when dealing with a pair of
dimensions that are both symbolic. Minimal example:

import hidet

if __name__ == "__main__":
    x = hidet.symbol(["n"])
    y = hidet.symbol(["m"])
    z = x + y
    print(x.shape, y.shape, z.shape) # before: (n,) (m,) (m,)
vadiklyutiy pushed a commit that referenced this pull request Jul 22, 2024
The previous implementation is incorrect when dealing with a pair of
dimensions that are both symbolic. Minimal example:

import hidet

if __name__ == "__main__":
    x = hidet.symbol(["n"])
    y = hidet.symbol(["m"])
    z = x + y
    print(x.shape, y.shape, z.shape) # before: (n,) (m,) (m,)
vadiklyutiy pushed a commit that referenced this pull request Jul 23, 2024
The previous implementation is incorrect when dealing with a pair of
dimensions that are both symbolic. Minimal example:

import hidet

if __name__ == "__main__":
    x = hidet.symbol(["n"])
    y = hidet.symbol(["m"])
    z = x + y
    print(x.shape, y.shape, z.shape) # before: (n,) (m,) (m,)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants