-
Notifications
You must be signed in to change notification settings - Fork 6.8k
updated news.md and readme.md for 1.8.0 release #19975
Conversation
Hey @samskalicky , Thanks for submitting the PR
CI supported jobs: [windows-cpu, centos-cpu, miscellaneous, website, clang, centos-gpu, sanity, unix-gpu, edge, unix-cpu, windows-gpu] Note: |
It's great to see 1.8.0 release, kudos to @samskalicky ! |
Honestly, I dont think we need this News.md updated as we have been doing. Its pretty much a duplication of whats in the release notes: https://github.com/apache/incubator-mxnet/releases and in the Readme.md we actually link to that page. The News.md isnt used anywhere anymore, users actually have to go click it to view it. it doesnt show up on the main GitHub page or the website anymore. |
- Backport TRT test update #19296 (#19298) | ||
#### OneDNN | ||
- Upgrade to oneDNN v1.6.3 (#19153) (#19161) | ||
- Update oneDNN to official v1.6 release (#18867) (#18867) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
duplicated PR number?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
updated news.md and readme.md for 1.8.0 release