Skip to content

[REFACTOR] Remove legacy nnvm folder#10821

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

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

Conversation

@tqchen
Copy link
Member

@tqchen tqchen commented Mar 29, 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.

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
Copy link
Member Author

tqchen commented Mar 29, 2022

@tqchen
Copy link
Member Author

tqchen commented Mar 29, 2022

cc @kparzysz-quic @szha

@szha
Copy link
Member

szha commented Mar 29, 2022

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
Copy link
Member Author

tqchen commented Mar 29, 2022

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

Comments