driazati commented on a change in pull request #9971:
URL: https://github.com/apache/tvm/pull/9971#discussion_r793945463
##########
File path: tests/scripts/task_python_docs.sh
##########
@@ -34,14 +32,78 @@ cleanup()
}
trap cleanup 0
-# cleanup old states
-rm -rf docs/_build
-rm -rf docs/_staging
-mkdir -p docs/_build/html
-mkdir -p docs/_staging/html
-rm -rf docs/gen_modules
-rm -rf docs/doxygen
+clean_files() {
+ # cleanup old states
+ rm -rf docs/_build
+ rm -rf docs/_staging
+ mkdir -p docs/_build/html
+ mkdir -p docs/_staging/html
+ rm -rf docs/gen_modules
+ rm -rf docs/doxygen
+ find . -type f -path "*.pyc" | xargs rm -f
+}
+
+sphinx_precheck() {
+ clean_files
+ echo "PreCheck sphinx doc generation WARNINGS.."
+ make cython3
+
+ pushd docs
+ make clean
+ TVM_TUTORIAL_EXEC_PATTERN=none make html 2>&1 | tee /tmp/$$.log.txt
+ check_sphinx_warnings "docs"
+ popd
+}
+
+
+function join_by { local IFS="$1"; shift; echo "$*"; }
+
+IGNORED_WARNINGS=(
+ '__mro__'
Review comment:
Added a comment, generally people shouldn't be adding new warnings here
unless there are infra problems (i.e. we had to ignore some stuff when bumping
up the PyTorch version). Most of these aren't new and were being ignored
before, just not in the actual docs build (where they weren't checked at all)
and only on the precheck.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
