Skip to content
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

๐Ÿ“ docs: Add docs for env LLM_VISION_IMAGE_USE_BASE64 #4977

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

CaiJingLong
Copy link
Contributor

@CaiJingLong CaiJingLong commented Dec 11, 2024

๐Ÿ’ป ๅ˜ๆ›ด็ฑปๅž‹ | Change Type

  • โœจ feat
  • ๐Ÿ› fix
  • โ™ป๏ธ refactor
  • ๐Ÿ’„ style
  • ๐Ÿ‘ท build
  • โšก๏ธ perf
  • ๐Ÿ“ docs
  • ๐Ÿ”จ chore

๐Ÿ”€ ๅ˜ๆ›ด่ฏดๆ˜Ž | Description of Change

if (type === 'url' && process.env.LLM_VISION_IMAGE_USE_BASE64 === '1') {
const { base64, mimeType } = await imageUrlToBase64(content.image_url.url);
return {
...content,
image_url: { ...content.image_url, url: `data:${mimeType};base64,${base64}` },
};
}

๐Ÿ“ ่กฅๅ……ไฟกๆฏ | Additional Information

Copy link

vercel bot commented Dec 11, 2024

The latest updates on your projects. Learn more about Vercel for Git โ†—๏ธŽ

Name Status Preview Comments Updated (UTC)
lobe-chat-database โœ… Ready (Inspect) Visit Preview ๐Ÿ’ฌ Add feedback Dec 12, 2024 1:06am
lobe-chat-preview โœ… Ready (Inspect) Visit Preview ๐Ÿ’ฌ Add feedback Dec 12, 2024 1:06am

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Dec 11, 2024
Copy link

vercel bot commented Dec 11, 2024

@CaiJingLong is attempting to deploy a commit to the LobeChat Community Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added the ๐Ÿ“ Documentation Improvements or additions to documentation | ๆ–‡ๆกฃ้—ฎ้ข˜ label Dec 11, 2024
@CaiJingLong CaiJingLong changed the title Docs--LLM_VISION_IMAGE_USE_BASE64 Docs: Add docs for env LLM_VISION_IMAGE_USE_BASE64 Dec 11, 2024
Copy link

codecov bot commented Dec 11, 2024

Codecov Report

All modified and coverable lines are covered by tests โœ…

Project coverage is 91.97%. Comparing base (aab0c53) to head (8a332ad).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff            @@
##             main    #4977    +/-   ##
========================================
  Coverage   91.97%   91.97%            
========================================
  Files         548      548            
  Lines       41357    41357            
  Branches     3706     3550   -156     
========================================
  Hits        38038    38038            
  Misses       3319     3319            
Flag Coverage ฮ”
app 91.97% <รธ> (รธ)
server 98.12% <รธ> (รธ)

Flags with carried forward coverage won't be shown. Click here to find out more.

โ˜” View full report in Codecov by Sentry.
๐Ÿ“ข Have feedback on the report? Share it here.

Copy link
Contributor

@arvinxx arvinxx left a comment

Choose a reason for hiding this comment

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

่ฟ™ไธชๅ˜้‡ๅปบ่ฎฎๆ”พๅˆฐ S3 ไธ‹้ขๅ“ˆ๏ผŒๅ› ไธบไธ€่ˆฌๅชๆœ‰้…็ฝฎ S3 ็š„ๆ—ถๅ€™ๆ‰้œ€่ฆ้…่ฟ™ไธช

@CaiJingLong
Copy link
Contributor Author

ๆˆ‘ๆ˜ฏๅœจ docker ้ƒจ็ฝฒ็š„ๆ—ถๅ€™็”จๅˆฐ็š„๏ผŒOK๏ผŒๆˆ‘ไผš็งปๅŠจๅˆฐ S3 ไธ‹

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Dec 12, 2024
@CaiJingLong CaiJingLong force-pushed the docs--LLM_VISION_IMAGE_USE_BASE64 branch from e521206 to 8a332ad Compare December 12, 2024 00:47
@arvinxx arvinxx changed the title Docs: Add docs for env LLM_VISION_IMAGE_USE_BASE64 ๐Ÿ“ docs: Add docs for env LLM_VISION_IMAGE_USE_BASE64 Dec 12, 2024
@arvinxx arvinxx merged commit a3fe8e2 into lobehub:main Dec 12, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
๐Ÿ“ Documentation Improvements or additions to documentation | ๆ–‡ๆกฃ้—ฎ้ข˜ size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants