-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Web UI manual ad annotation functionality #12
Comments
I wonder if another (potentially simpler) approach is to allow users to provide a list of known advertisers, in the hope of giving the LLM a greater chance of identifying ad segments. I've tweaked my
|
That definitely seems like it would help. If it does, maybe we could consider adding first class support for basically an I still think the web UI transcript thing is going to be helpful though. Even if the ad list is perfectly effective, some flow where you
is going to be much nicer in some form of UI |
As promised, just a quick follow-up: Updating my prompt to include a list of known advertisers has improved the detection of adverts that were previously being overlooked 🙌 Being able to update this list through a web interface would be a lovely feature. Thanks again for all your work! 👏 |
Add the ability to use a web UI to manually annotate ads.
Probably follows #11
This may end up being the same PR as #9
Basically:
Segment
The text was updated successfully, but these errors were encountered: