-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
|
big damn hero |
| @@ -0,0 +1,185 @@ | |||
| # PostHog Integration | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is so interesting: are you finding this forensic analysis is helping the agent make better calls?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's an attempt... I ended up just moving it to the README. I think this does help. But I think just having it in the readme is enough from testing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This worked super well at first because I tested on the same code as the examples which is obv a dumb dumb move by me. It was overfitting to all heck.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
HAHAHAHA it’s so easy to get tripped up in this specific fashion
daniloc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I say go for it when ready and we can surely iterate from here. but you've been so thorough, this can only improve things.
|
Gonna clean up and merge this in. Will tag y'all on the wizard PR soon |
Examples used for these docs