Skip to content

chore: update analyze examples to include the usage of json formatting#142

Merged
park-yeonhoo merged 3 commits intomainfrom
include-response-format-example
Sep 17, 2025
Merged

chore: update analyze examples to include the usage of json formatting#142
park-yeonhoo merged 3 commits intomainfrom
include-response-format-example

Conversation

@park-yeonhoo
Copy link
Contributor

@park-yeonhoo park-yeonhoo commented Sep 9, 2025

Summary

included examples for using response_format and max_token fields for /analyze endpoint

Testing done

source venv/bin/activate && API_KEY=*** PYTHONPATH=src python examples/analyze.py

@park-yeonhoo park-yeonhoo force-pushed the include-response-format-example branch from 562fa48 to f7b3a5a Compare September 9, 2025 21:28
Copy link
Contributor

@tl-genie tl-genie left a comment

Choose a reason for hiding this comment

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

did you forget to import response types? example looks good overall, but I think you need to regenerate SDK with the new schema via Fern generator (if needed)

@park-yeonhoo park-yeonhoo force-pushed the include-response-format-example branch from 0b1b745 to 4699e2b Compare September 17, 2025 05:22
@park-yeonhoo
Copy link
Contributor Author

did you forget to import response types? example looks good overall, but I think you need to regenerate SDK with the new schema via Fern generator (if needed)

addressed with new version that is release with fern

cursor[bot]

This comment was marked as outdated.

@park-yeonhoo park-yeonhoo force-pushed the include-response-format-example branch from f50faa6 to d7693e9 Compare September 17, 2025 05:53
Signed-off-by: park-yeonhoo <park@twelvelabs.io>
Signed-off-by: park-yeonhoo <park@twelvelabs.io>
Signed-off-by: park-yeonhoo <park@twelvelabs.io>
@park-yeonhoo park-yeonhoo force-pushed the include-response-format-example branch from d7693e9 to 8003ff8 Compare September 17, 2025 05:55
@park-yeonhoo park-yeonhoo requested a review from a team September 17, 2025 17:59
@andreipope andreipope self-requested a review September 17, 2025 18:05
Copy link
Collaborator

@andreipope andreipope left a comment

Choose a reason for hiding this comment

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

Looks good!

@park-yeonhoo park-yeonhoo merged commit 05305b7 into main Sep 17, 2025
5 checks passed
@park-yeonhoo park-yeonhoo deleted the include-response-format-example branch September 17, 2025 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments