-
Notifications
You must be signed in to change notification settings - Fork 2
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
Assistant add Gauge UI to machine generated text #686
base: beta-master
Are you sure you want to change the base?
Conversation
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 looks very nice! 😍
Hi @rosannamilner, this looks good! Did you get any feedback from KInIT on this, especially on the probability scores, the scale and their meanings? Partners in Vera often have different wordings and / or scales for the results of their models and I just want to check that they have been consulted before merging this |
No I haven't, good point - I will do this now! |
Kalina has suggested removing the bottom "Hide detection details" entirely as it's duplicated information in this case. Also, there is extra whitepace around the gauge as compared to in Deepfake Image/Video as the image/video would sit on the left hand side of the GaugeChartResult component. @Sallaa Please let me know your thoughts on this and I can change the UI now, or we can leave this for a future improvement down the line if not too important now? |
Yes, it makes sense to remove the details if it does not add any information to the user. I'm also fine if the UI is not pixel perfect. We can ship it as it is (once you've collected KInIT feedback and scale) and adjust it later as we also collect feedback from users |
To go with InVID Translations PR: AFP-Medialab/InVID-Translations#109 |
@rosannamilner LGTM -- I will merge this once you have resolved the conflicts for the translation PR |
Thanks - resolved |
Adding GaugeChartResult to Machine Generated Text
arcsLengths
: I can't seem to get the rightDETECTION_THRESHOLDS
for the arcs I need to display, so I have added sending predefinedarcsLengths
as propsDETECTION_THRESHOLDS = { THRESHOLD_1: 5.0, THRESHOLD_2: 50.0, THRESHOLD_3: 95.0, }
arcsLength = [0.05, 0.45, 0.45, 0.05]
detectionType = "machine_generated_text"
align="start"
as text was centeredQuestions
Example link: https://www.breitbart.com/news/suspect-arrested-after-hunt-for-us-insurance-chiefs-killer/