Skip to content
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

add buttons to rate logs #4629

Merged
merged 1 commit into from
Oct 13, 2023
Merged

add buttons to rate logs #4629

merged 1 commit into from
Oct 13, 2023

Conversation

PaliC
Copy link
Contributor

@PaliC PaliC commented Oct 11, 2023

add buttons to rate logs

Adds buttons hidden under a feature flag which gives us the ability to rate logs
Screenshot 2023-10-11 at 4 55 55 PM

🤖 Generated by Copilot at e97f54f

This pull request adds the feature to annotate the log rating of a job in the LogViewer component. It introduces a new component LogAnnotationToggle that allows users to select an annotation and sends it to a new API handler /api/log_annotation. It also defines a new type LogAnnotation and stores the annotation data in a DynamoDB table.


Stack created with Sapling. Best reviewed with ReviewStack.

@vercel
Copy link

vercel bot commented Oct 11, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
torchci ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 13, 2023 9:55pm

Adds buttons hidden under a feature flag which gives us the ability to rate logs
<img width="1444" alt="Screenshot 2023-10-11 at 4 55 55 PM" src="https://github.com/pytorch/test-infra/assets/13758638/76748be5-44f7-4e51-8e72-d781a96e9ba3">

<!--
copilot:summary
-->
### <samp>🤖 Generated by Copilot at e97f54f</samp>

This pull request adds the feature to annotate the log rating of a job in the `LogViewer` component. It introduces a new component `LogAnnotationToggle` that allows users to select an annotation and sends it to a new API handler `/api/log_annotation`. It also defines a new type `LogAnnotation` and stores the annotation data in a DynamoDB table.
@PaliC PaliC merged commit 94a36cb into main Oct 13, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants