gemini-code-assist[bot] commented on code in PR #18925:
URL: https://github.com/apache/tvm/pull/18925#discussion_r2978193155
##########
python/tvm/contrib/pickle_memoize.py:
##########
@@ -71,6 +71,14 @@ def cache(self):
if self.path.exists():
with self.path.open("rb") as cache_file:
try:
+ import warnings
Review Comment:
The `import warnings` statement should be moved to the top of the file.
According to PEP 8, imports should be at the top of the file, just after any
module comments and docstrings. Placing imports inside functions is generally
discouraged as it makes it harder to see the module's dependencies at a glance
and can lead to repeated imports.
<details>
<summary>References</summary>
1. PEP 8 recommends that imports are always put at the top of the file, just
after any module comments and docstrings, and before module globals and
constants.
<sup>([link](http://console.cloud.google.com/gemini-code-assist/agents-tools))</sup>
</details>
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
