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 Slackbot #420

Merged
merged 1 commit into from
Nov 13, 2024
Merged

Add Slackbot #420

merged 1 commit into from
Nov 13, 2024

Conversation

gordlin
Copy link
Member

@gordlin gordlin commented Nov 12, 2024

Changes

  • Adds a Slack bot that posts every time a new PR is created in the storylines-editor repo, giving details of the PR as well as an AI-generated summary of its description.
    • The AI summary uses the Pegasus summarizer model, which seems to do a good job and requires significantly fewer compute resources than a generalist model (ala LLAMA 3.2).
    • The Python transformers library is used to handle the model.

Notes

Links to the PR types list pages ("PR: Active", "PR: Geo", etc.) are hard-coded, and will need to be manually changed if the names of the types are modified.
QA Testing
Please test against main branch https://ramp4-pcar4.github.io/ramp4-pcar4/main/demos/enhanced-samples.html

Testing

  1. Read the YAML file, and ensure the workflow is correct.
  2. Once merged, the bot should summarize all new PRs in the storylines-editor repo.

This change is Reviewable

@gordlin gordlin added PR: Active PRs that require a fierce eyeballin. PR: Build PR that involves changes to the build. This includes changes to the Github Actions. For JS wizards. labels Nov 12, 2024
Copy link

Your demo site is ready! 🚀 Visit it here: https://ramp4-pcar4.github.io/storylines-editor/slackbot

Copy link
Member

@szczz szczz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @gordlin)

@szczz szczz merged commit 483a170 into ramp4-pcar4:main Nov 13, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Active PRs that require a fierce eyeballin. PR: Build PR that involves changes to the build. This includes changes to the Github Actions. For JS wizards.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants