Closed evamaxfield closed 2 years ago
Further thinking about this, maybe we should only show this on the event page if the transcript was generated by a known ML method. We store the generator in the transcript file: https://github.com/CouncilDataProject/cdp-backend/blob/main/cdp_backend/pipeline/transcript_model.py#L174
If the generator matches a regex like: https://github.com/CouncilDataProject/cdp-backend/blob/main/cdp_backend/sr_models/google_cloud_sr_model.py#L248
Then display this disclaimer?
Hey @JacksonMaxfield , is anyone working on this, and if not, can I pick it up? Mark is in contact with our city about our CDP instance and I think it'd be good to have this disclaimer since all of our transcripts are ML generated.
To my knowledge no one is working on this! Feel free to take it on.
Totally agree that it's a good thing to have haha. Where to put it on the single event page / maybe search page too is the question in my mind.
@JacksonMaxfield Cool, I'll try a few placements out and see what y'all think!
Wooo! Thanks!
I made a prototype of a few options, and I want to get early feedback:
I can add a box around the text, color the text, or change the font size. I can also put the text below the meetings.
For now, the text is displayed for all results. Once we decide how to display the text, I will how to filter it only for ML classifications. I think that classification can only work for the first option since we have too many events in the second option—some of them can be ML-generated, others can be human-generated.
I am open to receiving any advice.
Hey @Shak2000 these look good, I have one suggestion but before I get to it, I am curious if you can try deploying: https://github.com/CouncilDataProject/cdp-frontend/blob/main/CONTRIBUTING.md#deploying-your-storybook-docs-site-or-example-app
Specifically:
npm run build:app
npm run deploy:app
Then we can simply go to your page and check it out.
Here it is: https://shak2000.github.io/cdp-frontend/#/
Thanks! I think I like the event search results location more than the transcript search results location.
Feel free to remove the transcript search results one.
I am tempted to ask to also place this in the footer so it's always present.
I may have some general rewording later
I removed the transcript search location. I added the footer. I put the footer where the copyrights. I can move it into the links in the area above
I removed the transcript search location. I added the footer. I put the footer where the copyrights. I can move it into the links in the area above
Wanna open a PR?
Four things:
I implemented all of the requests
Can you please assign it to me? I have a PR waiting
We have known there will always be some transcription errors and until we invest in making our own model, we should at the very least put a disclaimer about them: