- Issue created by @murz
- @murz opened merge request.
- π¦π²Armenia murz Yerevan, Armenia
I created two base classes:
- AiProviderRequestBaseEvent
- AiProviderResponseBaseEvent
that stores all the common properties and getters-setters for the child events - this allowed us to make the real event classes compact and almost empty: + 521 β 819 lines ;)Moving the issue to needs review.
- π©πͺGermany a.dmitriiev
a.dmitriiev β made their first commit to this issueβs fork.
- π©πͺGermany a.dmitriiev
The idea is good and the implementation looks also fine. I have only one concern, please check the MR comment.
- π¦π²Armenia murz Yerevan, Armenia
So, if the current implementation is okay, let's merge this then? Rebased on the fresh base branch, please make a fresh review.
- π©πͺGermany a.dmitriiev
It looks good now, but there is now problem in
src/Base/OpenAiBasedProviderClientBase.php
line 356, thesetChatTokenUsage
method expects second parameter to be array, butStreamedResponse
object is passed. Maybe Marcus can check this?Because of this error there is no way to test the Streamed logging.
- π¦π²Armenia murz Yerevan, Armenia
Seems the problem was in the wrong order of parameters, I fixed this and reworked the code to use named parameters to minimize such errors in the future, take a look pls.