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

Create llm.ytag #158

Merged
merged 6 commits into from
May 3, 2024
Merged

Create llm.ytag #158

merged 6 commits into from
May 3, 2024

Conversation

ekulxam
Copy link
Contributor

@ekulxam ekulxam commented Apr 30, 2024

A tag to tell people not to write code with chatgpt.

I didn't do a very good job of phrasing this one, so feel free to edit it.


---

LLM is bad and inconsistent at writing code. Do not use ChatGPT or other large language models to try to write mods. Please learn java first.
Copy link
Member

Choose a reason for hiding this comment

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

LLM is bad

I disagree, its a great tool if used right. Sadly people expect it to just write a whole mod in one go. I think we should have a tag that explains why it might not be writing perfect code for them.

Copy link
Member

@IMB11 IMB11 left a comment

Choose a reason for hiding this comment

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

Potentially this is a good rewrite?

tags/faq/llm.ytag Outdated Show resolved Hide resolved
@ekulxam
Copy link
Contributor Author

ekulxam commented Apr 30, 2024

Updated tag with IMB11's suggested changes

@Poopooracoocoo
Copy link
Contributor

Looks great. Some aliases for the tag such as chatgpt, copilot, and Gemini might be useful? Not very important to have aliases tho

@ekulxam
Copy link
Contributor Author

ekulxam commented May 1, 2024

Added chatgpt, copilot, and gemini as aliases.

@ekulxam ekulxam requested review from modmuss50 and IMB11 May 1, 2024 03:26
@modmuss50 modmuss50 merged commit 99468ef into FabricMC:main May 3, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants