Skip to content

[Bugfix] fix the invalid image path#2899

Merged
DongDongJu merged 4 commits intoLMCache:devfrom
SYaoJun:fix_image
Mar 30, 2026
Merged

[Bugfix] fix the invalid image path#2899
DongDongJu merged 4 commits intoLMCache:devfrom
SYaoJun:fix_image

Conversation

@SYaoJun
Copy link
Copy Markdown
Contributor

@SYaoJun SYaoJun commented Mar 28, 2026

What this PR does / why we need it:
image path is invalid
image

Special notes for your reviewers:

If applicable:

  • this PR contains user facing changes - docs added
  • this PR contains unit tests

Signed-off-by: syaojun <libevent@yeah.net>
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the image path for the LMCache Controller diagram in the documentation. A suggestion was made to use an absolute path from the Sphinx source root to improve maintainability and prevent potential link breakage during future file restructuring.

Comment thread docs/source/kv_cache_management/index.rst Outdated
@SYaoJun SYaoJun changed the title fix the invalid image path [Bugfix] fix the invalid image path Mar 28, 2026
Copy link
Copy Markdown
Collaborator

@deng451e deng451e left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@deng451e deng451e requested a review from sammshen March 29, 2026 01:05
SYaoJun and others added 2 commits March 30, 2026 09:24
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Jason Yao <libevent@yeah.net>
Copy link
Copy Markdown
Collaborator

@DongDongJu DongDongJu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@DongDongJu DongDongJu enabled auto-merge (squash) March 30, 2026 07:07
@deng451e deng451e added the full Run comprehensive tests on this PR label Mar 30, 2026
@DongDongJu DongDongJu merged commit d2a8596 into LMCache:dev Mar 30, 2026
34 checks passed
jooho-XCENA pushed a commit to xcena-dev/LMCache that referenced this pull request Apr 2, 2026
* fix the invalid image path

Signed-off-by: syaojun <libevent@yeah.net>

* Apply suggestion from @gemini-code-assist[bot]

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Jason Yao <libevent@yeah.net>

---------

Signed-off-by: syaojun <libevent@yeah.net>
Signed-off-by: Jason Yao <libevent@yeah.net>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
jooho-XCENA pushed a commit to xcena-dev/LMCache that referenced this pull request Apr 2, 2026
* fix the invalid image path

Signed-off-by: syaojun <libevent@yeah.net>

* Apply suggestion from @gemini-code-assist[bot]

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Jason Yao <libevent@yeah.net>

---------

Signed-off-by: syaojun <libevent@yeah.net>
Signed-off-by: Jason Yao <libevent@yeah.net>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full Run comprehensive tests on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants