Skip to content

Commit c477b30

Browse files
committed
fix up according to review comments
Signed-off-by: SamYuan1990 <yy19902439@126.com>
1 parent 39969a8 commit c477b30

File tree

3 files changed

+9
-1
lines changed

3 files changed

+9
-1
lines changed

examples/clients/homebrewupload-client/README.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,9 @@ See `examples/servers/homebrewupload`
2424
# Navigate to the server directory
2525
cd examples/clients/homebrewupload-client
2626

27+
# You need to make a pdf file as test.pdf
28+
# examples/clients/homebrewupload-client/test.pdf
29+
2730
## defualt tested with DeepSeek as LLM provider
2831
export ANTHROPIC_BASE_URL=https://api.deepseek.com/anthropic
2932
export ANTHROPIC_API_KEY=<Your_API_KEY>
@@ -32,4 +35,9 @@ export ANTHROPIC_API_KEY=<Your_API_KEY>
3235
uv run python main.py
3336
```
3437

38+
## Token cosumption discussion
3539

40+
In auther's local test, auther use a pdf with content as `hello world`.
41+
When using data style URI `data...base64...` it consumes about 30k token after base64.
42+
When using this example, using `file:...path...` instead.
43+
Which impls execution out of LLM context, just consumes about token on file path and `hello world`.
-23.2 KB
Binary file not shown.

examples/servers/homebrewupload/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,4 +27,4 @@ move to `examples/clients/homebrewupload-client`
2727

2828
## Further consideration
2929

30-
As if we running it as container, then on k8s, we can use service mesh and etc to handle with security as AA.
30+
As if we running it as container, then on k8s, we can use service mesh and etc to handle with security as AA.

0 commit comments

Comments
 (0)