gemini-code-assist[bot] commented on code in PR #193:
URL: https://github.com/apache/tvm-ffi/pull/193#discussion_r2464185806


##########
docs/get_started/stable_c_abi.rst:
##########
@@ -79,7 +79,7 @@ Layout of :cpp:class:`TVMFFIAny`
 - Next 32 bits: reserved (used for flags in rare cases, e.g. small-string 
optimization).
 - Last 64 bits: payload that is either a 64-bit integer, a 64-bit 
floating-point number, or a pointer to a heap-allocated object.
 
-.. figure:: 
https://raw.githubusercontent.com/tlc-pack/web-data/refs/heads/main/images/tvm-ffi/stable-c-abi-layout-any.svg
+.. figure:: 
https://raw.githubusercontent.com/tlc-pack/web-data/main/images/tvm-ffi/stable-c-abi-layout-any.svg?sanitize=true

Review Comment:
   ![medium](https://www.gstatic.com/codereviewagent/medium-priority.svg)
   
   While adding `?sanitize=true` fixes the rendering, a more robust approach is 
to store the images within this repository, for example in a 
`docs/_static/images/` directory. This makes the documentation self-contained, 
removes the dependency on `raw.githubusercontent.com`, and avoids potential 
issues with GitHub's file serving policies in the future. You could then 
reference the image with a relative path, for example: `.. figure:: 
/_static/images/tvm-ffi/stable-c-abi-layout-any.svg`.



-- 
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]

Reply via email to