-
Notifications
You must be signed in to change notification settings - Fork 25
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
[libshortfin] Enable building docs with Sphinx #209
Conversation
@@ -0,0 +1,36 @@ | |||
# Configuration file for the Sphinx documentation builder. |
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.
add license/copyright header?
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.
Added a copyright header to the files.
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.
Can they use the standard format? https://github.com/nod-ai/SHARK-Platform/blob/19834ee1e1d476a116679afb9521ea0cccdeb308/shortfin/shortfin/framework/session.py#L1-L5
# Copyright 2024 Advanced Micro Devices, Inc.
#
# Licensed under the Apache License v2.0 with LLVM Exceptions.
# See https://llvm.org/LICENSE.txt for license information.
# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
# -- Options for HTML output ------------------------------------------------- | ||
# https://www.sphinx-doc.org/en/master/usage/configuration.html#options-for-html-output | ||
|
||
html_theme = "sphinx_rtd_theme" |
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.
Did you try other themes? Wonder if there is something that AMD recommends
https://rocm.docs.amd.com/en/latest/contribute/toolchain.html points to https://sphinx-book-theme.readthedocs.io/en/latest/
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.
No not yet. I wasn't able to figure out if there is some common theme used and that's at least in line with the IREE Python API docs.
012a9ac
to
852df14
Compare
@@ -0,0 +1,36 @@ | |||
# Configuration file for the Sphinx documentation builder. |
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.
Can they use the standard format? https://github.com/nod-ai/SHARK-Platform/blob/19834ee1e1d476a116679afb9521ea0cccdeb308/shortfin/shortfin/framework/session.py#L1-L5
# Copyright 2024 Advanced Micro Devices, Inc.
#
# Licensed under the Apache License v2.0 with LLVM Exceptions.
# See https://llvm.org/LICENSE.txt for license information.
# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
852df14
to
3325ac5
Compare
No description provided.