Peter Goldstein
Peter Goldstein
@johannesstricker Did you open a PR? We're also interested in Bedrock support (Claude 3 in particular) and have dev effort to contribute, but wouldn't want to duplicate effort.
@johannesstricker Does #847, which was merged, address this issue in your opinion? Claude 3 (and shortly Claude 3.5) models are supported. Other models on Bedrock can be added relatively easily...
Is this still desired now that Gemini support is in? The PaLM API is deprecated as I understand it - https://ai.google.dev/palm_docs/deprecation
I like this change, and I don't think a slight performance impact on older rubies is a material concern.
@grcooper Do we know why we're seeing repeated test failures?
@nirvdrum So I would, provided there's someone on the TruffleRuby side who steps up to be a point of contact for issues. I've seen TruffleRuby issues languish in other repos...
@olleolleolle So is this an alternative to #1008 ? Are there any concerns on how this works with older Ruby versions? Will they see a meaningful change in behavior?
@olleolleolle It would helpful to understand how these different PRs relate, since they seem to touch similar code.
@olleolleolle can you please address the Rubocop warnings? Thanks.
@nickamorim I agree that the current behavior is problematic, and we should adjust with what amounts to a breaking change. How does this behave when cache_nils is true? I'm not...