-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DOCS][ADRENO] Improved Adreno documentation #13867
Conversation
c14b573
to
772ce6f
Compare
Unified single documentation for all types of usage with OpenCL as well as CLML backends. Detailed simplified usage (with docker environment and command line tools like tvmc) as well as advanced usage instructions via python based interface.
772ce6f
to
0e05122
Compare
Co-authored-by: Egor Churaev <[email protected]>
Co-authored-by: Egor Churaev <[email protected]>
3502ccf
to
a43658d
Compare
a43658d
to
d4c1c7c
Compare
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.
Several comments
4c24470
to
2cdf5fe
Compare
2cdf5fe
to
3f9609b
Compare
Co-authored-by: Egor Churaev <[email protected]>
Co-authored-by: Egor Churaev <[email protected]>
49b4bbb
to
caec976
Compare
caec976
to
ad82f8c
Compare
f3ea881
to
b2bb23b
Compare
9fe4cb8
to
5b9b1f2
Compare
5b9b1f2
to
f6e9a4d
Compare
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.
Several minor comments
Co-authored-by: Egor Churaev <[email protected]>
Co-authored-by: Egor Churaev <[email protected]>
"app") | ||
def_environment | ||
echo "Setting dev environment with Tracker Port : $TVM_TRACKER_HOST} and the available devices are" | ||
python3 -m tvm.exec.query_rpc_tracker --port ${TVM_TRACKER_PORT} |
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.
As far as I understood, the app
mode is only for printing the list of the available devices (just call query), am I right?
Probably we should rename this mode to query
? And you didn't describe this mode in the documentation. Could you please add it?
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.
Its sets the pytest and python environments.
340caef
to
908d8e0
Compare
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. Thank you @srkreddy1238 for hard working on this PR. The final result looks incredible. Hope it will be very helpful for new users.
Unified single documentation for all types of usage with OpenCL as well as CLML backends.
Detailed simplified usage (with docker environment and command line tools like tvmc) as well as advanced usage instructions via python based interface.