-
Notifications
You must be signed in to change notification settings - Fork 3
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
[curator-viewer] add time logging and curator viewer show distribution #149
[curator-viewer] add time logging and curator viewer show distribution #149
Conversation
let me add one more view, which is the time series plot, that's useful for #141 comparison with OpenAIOnline |
Ready for review @vutrung96 @RyanMarten |
Some errors I found:
|
point1, 2 resolved. need to double check whether point3 is actually been resolved, can't reproduce the same setting from my end. |
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.
can confirm it works on my end and the graph looks beautiful. thank you @CharlieJCJ !
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.
Changes:
create_by
,finished_by
#145New GenericResponse:
Screen.Recording.2024-11-19.at.4.15.49.PM.mov
Screen.Recording.2024-11-19.at.2.05.59.PM.mov
Nice to have: