Skip to content

Conversation

@mshr-h
Copy link
Contributor

@mshr-h mshr-h commented Jun 6, 2024

Without this fix,

TVMError: Check failed: (min->IsInstance<PrimValueNode>()) is false: The argument `min` of relax.clip is expected to be a PrimValue, but gotrelax.expr.Constant

With this fix,

TVMError: Check failed: (min->IsInstance<PrimValueNode>()) is false: The argument `min` of relax.clip is expected to be a PrimValue, but got relax.expr.Constant

cc: @yongwww @tqchen @lhutton1

@mshr-h
Copy link
Contributor Author

mshr-h commented Jun 6, 2024

@tvm-bot rerun

@mshr-h mshr-h changed the title [Relax] Add missing white spaces [Relax] Add missing white spaces in error messages Jun 6, 2024
@mshr-h mshr-h marked this pull request as ready for review June 6, 2024 13:47
Copy link
Member

@yongwww yongwww 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

@quic-sanirudh quic-sanirudh merged commit 57914ff into apache:main Jun 7, 2024
@mshr-h mshr-h deleted the patch-1 branch June 10, 2024 01:01
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.

3 participants