Skip to content

[Relay][Op] Register some forgotten op in Python side#12963

Merged
junrushao merged 1 commit intoapache:mainfrom
MasterJH5574:bugfix/2022-10-01-relay-op-python
Oct 2, 2022
Merged

[Relay][Op] Register some forgotten op in Python side#12963
junrushao merged 1 commit intoapache:mainfrom
MasterJH5574:bugfix/2022-10-01-relay-op-python

Conversation

@MasterJH5574
Copy link
Contributor

It is noticed that many Attrs declared in this file are not registered on Python side. This will cause these Attrs having type Object when running with Python.

Therefore, this PR registers the missing operators in op_attrs.py, to make them have right types during runtime.

cc @spectrometerHBH @vinx13

Copy link
Member

@junrushao junrushao left a comment

Choose a reason for hiding this comment

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

LGTM

@junrushao junrushao merged commit c3357f6 into apache:main Oct 2, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
It is noticed that many Attrs declared in this [file](https://github.com/apache/tvm/blob/main/include/tvm/relay/attrs/transform.h) are not registered [on Python side](https://github.com/apache/tvm/blob/main/python/tvm/relay/op/op_attrs.py). This will cause these Attrs having type `Object` when running with Python.

Therefore, this PR registers the missing operators in `op_attrs.py`, to make them have right types during runtime.
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