Stuart Cam
Stuart Cam
Is it worth considering https://opentelemetry.io/ ?
@microsoft-github-policy-service agree
I have rebased again. Would it be possible to merge this sooner, rather than later, if possible? Thanks.
5th rebase (!)
Closing in favour of a new PR with "Enable edits from Maintainers" enabled.
Recreated from: https://github.com/microsoft/semantic-kernel/pull/738 Although I have now discovered, it is [not possible to allow edits by maintainers, since the fork exists with in organisation](https://github.com/orgs/community/discussions/5634).
Ping: @lemillermicrosoft
> Still see some formatting issues and build issues. Waiting on that for further testing. Is there anything specifically I can do to have this merged - I have opened...
Hi @dmytrostruk, Thanks for your review. I've pushed back on some of your requested changes and provided reasoning. The other requested changes I have made here: https://github.com/microsoft/semantic-kernel/pull/1219/commits/68145b525d78e4871e40064b8afe058a837546df Please take another...
@dmytrostruk - please can you review? I have removed the `SK` prefix and wrapped every possible exception with the `WeaviateMemoryException` class. If this is the desired behaviour, then I would...