AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

fix(ops/issues): update template for bugs

Open ntindle opened this issue 1 year ago • 8 comments

User description

Background

We added more LLM Providers, so we should support them in the bug template

Changes 🏗️

Adds new providers to the bug list

PR Quality Scorecard ✨

  • [x] Have you used the PR description template?   +2 pts
  • [ ] Is your pull request atomic, focusing on a single change?   +5 pts
  • [ ] Have you linked the GitHub issue(s) that this PR addresses?   +5 pts
  • [ ] Have you documented your changes clearly and comprehensively?   +5 pts
  • [ ] Have you changed or added a feature?   -4 pts
    • [ ] Have you added/updated corresponding documentation?   +4 pts
    • [ ] Have you added/updated corresponding integration tests?   +5 pts
  • [ ] Have you changed the behavior of AutoGPT?   -5 pts
    • [ ] Have you also run agbenchmark to verify that these changes do not regress performance?   +10 pts

PR Type

documentation


Description

  • Updated the bug report template to reflect the addition of new LLM providers.
  • Changed the dropdown label from "Do you use OpenAI GPT-3 or GPT-4?" to "What LLM Provider do you use?".
  • Added "Anthropic" and "Llamafile" as options in the LLM provider dropdown.

Changes walkthrough 📝

Relevant files
Documentation
1.bug.yml
Update bug template to include new LLM providers                 

.github/ISSUE_TEMPLATE/1.bug.yml

  • Updated the label for the LLM provider dropdown.
  • Added new LLM providers: Anthropic and Llamafile.
  • +3/-1     

    💡 PR-Agent usage: Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    ntindle avatar Jun 13 '24 02:06 ntindle

    PR Description updated to latest commit (https://github.com/Significant-Gravitas/AutoGPT/commit/71f894024fac533cd477f98eba612126bc75c2b4)

    qodo-merge-pro[bot] avatar Jun 13 '24 02:06 qodo-merge-pro[bot]

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review [1-5]

    1

    🧪 Relevant tests

    No

    🔒 Security concerns

    No

    ⚡ Key issues to review

    None

    qodo-merge-pro[bot] avatar Jun 13 '24 02:06 qodo-merge-pro[bot]

    Deploy Preview for auto-gpt-docs canceled.

    Name Link
    Latest commit fb66b4ae81c798effa4e37edbe42a95049bbf543
    Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/669e4db13bfe650008d49f34

    netlify[bot] avatar Jun 13 '24 02:06 netlify[bot]

    PR Code Suggestions ✨

    No code suggestions found for PR.

    qodo-merge-pro[bot] avatar Jun 13 '24 02:06 qodo-merge-pro[bot]

    Changed scope from docs to ops/issues. Lmk what you think.

    Pwuts avatar Jun 13 '24 17:06 Pwuts

    sounds good

    ntindle avatar Jun 13 '24 22:06 ntindle

    Add commit hash to the requirement for this - @Torantulino

    ntindle avatar Jun 14 '24 12:06 ntindle

    We also have Groq provider

    kcze avatar Jun 25 '24 16:06 kcze

    @ntindle Can we get this merged or closed today. It's be lying around for a while.

    I see there a few comments on what could be done.

    As long as its better than what we currently have, it should be merged and another PR can be made when there is time to improve on this.

    Swiftyos avatar Jul 22 '24 07:07 Swiftyos