Skip to content

[REFACTOR] Remove legacy nnvm folder#10821

Merged
masahi merged 1 commit into
apache:mainfrom
tqchen:nnvm
Mar 30, 2022
Merged

[REFACTOR] Remove legacy nnvm folder#10821
masahi merged 1 commit into
apache:mainfrom
tqchen:nnvm

Conversation

@tqchen

@tqchen tqchen commented Mar 29, 2022

Copy link
Copy Markdown
Member

nnvm was the first generation IR that was maintained by TVM before the community moved to a newer generation.
The community keeps it for a few more releases(v0.7 and v0.8).

This PR removes the folder. The source code can still be found in past releases.

nnvm was the first generation IR that was maintained by TVM before the community moved to a newer generation.
The community keeps it for a few more releases(v0.7 and v0.8).

This PR removes the folder. The source code can still be found in past releases.
@tqchen

tqchen commented Mar 29, 2022

Copy link
Copy Markdown
Member Author

@tqchen

tqchen commented Mar 29, 2022

Copy link
Copy Markdown
Member Author

cc @kparzysz-quic @szha

@szha

szha commented Mar 29, 2022

Copy link
Copy Markdown
Member

Thanks. @tqchen would you also open a PR to MXNet v1.X and master branches to replace the NNVM submodule with the actual code?

@tqchen

tqchen commented Mar 29, 2022

Copy link
Copy Markdown
Member Author

Thanks @szha. The repo should be fine as they already point to a past release which still contain the code. Please feel free to send the PR to mxnet sooner.

@masahi masahi merged commit f5c71c5 into apache:main Mar 30, 2022
pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
nnvm was the first generation IR that was maintained by TVM before the community moved to a newer generation.
The community keeps it for a few more releases(v0.7 and v0.8).

This PR removes the folder. The source code can still be found in past releases.
@tqchen tqchen deleted the nnvm branch February 26, 2023 13:55
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