bugfix: 291 - llm_tracker is not an attribute in client class

[andkoma]

📘 Description
initialize the llm_tracker in class client with None

🔄 Related Issue (if applicable)
#291

🎯 Goal
fix the bug, that the client class has in some cases no attribute for llm_tracker.

🔍 Additional Context
Looks like the class init is not complete processed and the attribute did not been created in some circumstances?

🧪 Testing
Fix in local environment, re-run the crewai application with agentops in two modes.
a) enabled
b) disabled.

[areibman]

Thanks for this! @bboynton97 what do you think?

[bboynton97]

Thank you for the contribution @andkoma! I think there's an underlying issue here we need to investigate, but this does prevent the case for an exception halting the execution of an agent due to AgentOps.

@areibman I say we merge this and investigate the issue after

[bboynton97]

@andkoma Could you merge upstream and run black .

If you dont have Black installed, run:

pip install black 'black[jupyter]'

[andkoma]

Ok. My change was just the insert of a single line statement. This could not be the root cause for the merge result. Regards

…

Am 29.07.2024 um 21:51 schrieb Braelyn Boynton ***@***.***>: @andkoma <https://github.com/andkoma> Could you merge upstream and run black . If you dont have Black installed, run: pip install black pip install black[jupyter] — Reply to this email directly, view it on GitHub <#320 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABYFEB24LDSAGC7YGQCND3LZO2MK5AVCNFSM6AAAAABLRYN7NKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJWG43TIMRYGY>. You are receiving this because you were mentioned.

[bboynton97]

There's no merge conflict. Your branch is behind and not formatted properly

[andkoma]

I just checked this behavior, I’ve made the change inside the web interface of the forked repo. Maybe in this case the embedded editor destroys your expected formatting. I will drop the fork and create a fresh one. Regards and sorry for any circumstances Andrew

…

Am 29.07.2024 um 23:31 schrieb Braelyn Boynton ***@***.***>: Merged #320 <#320> into main. — Reply to this email directly, view it on GitHub <#320 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABYFEBZPNUAHBQJQDQOMKIDZO2YB5AVCNFSM6AAAAABLRYN7NKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGY4DMOJRHE2TSOA>. You are receiving this because you were mentioned.