wenjin272 commented on PR #965: URL: https://github.com/apache/flink-agents/pull/965#issuecomment-5365021692
Thanks for addressing the previous comments @weiqingy. I reviewed the latest changes, and the PR looks good now. During the review, I noticed a separate pre-existing issue: the Anthropic integration defaults `temperature` to `0.1` and sends it with requests, while newer models such as Claude 4.7 and Mythos Preview reject non-default temperature settings. Since this behavior was not introduced by this PR, it should not block the PR and can be tracked separately. LGTM. -- 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]
