arnavsinghvi11

Thanks @DSamuelHodge for raising the issue and @sebbyjp for the fix! @sebbyjp could you actually create a separate PR for this change just to separate it from the rest of...

Hi @paulacanva , could you share the full error and stack trace if possible. `Connection Error` likely has to do with the configuration of the LM. Is `config.COMPLETION_CONFIG["model_teacher"]` a string?

Could you attach the full `ConnectionError` here? Additionally, are you configuring the LM using `dspy.settings.configure(lm=...)`?

Hi @paulacanva , it still seems to be like an error on the LM configuration end, potentially with connections or rate-limits (I found this issue to be potentially related to...

Thanks @xdite . Yes ideally, we can remove these definitions and have users define all the model kwargs directly (some models are not compatible with frequency_penalty, presence_penalty, etc.) but tagging...

Hi @rawsh , thanks for the contributions! I believe this may break the cache but tagging @okhat for reference here. Would it be possible to not touch the cache functions...

yeah ideally if we can add this support without modifying the existing behavior, this won't break the existing caches. Curious if you can only add the client creation as a...

@harrysalmon happy to merge this PR if ready!

Hi @harrysalmon , just following up again if this PR is ready to merge!

@amithmathew Does the merged #616 cover this?