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

feat: adding new feature to optimize apps #241

Closed
wants to merge 3 commits into from

Conversation

onedionys
Copy link

  • Adding new yaml

@aredridel
Copy link
Owner

Hello! Can you please tell me a bit of background on these changes?

@onedionys
Copy link
Author

Hello! Can you please tell me a bit of background on these changes?

Hello, I added that code to speed up the use of node-bin-gen. I realized it when using node-bin-gen because it takes a while.
You can approve the PR or not, feel free. However, I recommend approving it so that it can be useful for others who use node-bin-gen. Thanks for your feedback!

@aredridel
Copy link
Owner

And how does it speed it up?

@aredridel
Copy link
Owner

I believe this PR is LLM-generated spam trying to integrate the pkgx project.

@aredridel aredridel closed this Feb 26, 2024
@onedionys
Copy link
Author

I believe this PR is LLM-generated spam trying to integrate the pkgx project.

Yep, you're right. Actually, I only want to add one file, tea.yml, to your repository. Because I have a job that requires uploading the file and I also don't know what it is used for.
I apologize and thank you 😓🙏

@oldherl
Copy link

oldherl commented Feb 26, 2024

This is a crypto scam and the PR author is trying to steal potential tokens from you.
https://connortumbleson.com/2024/02/26/the-disappointing-tea-xyz/

@aredridel
Copy link
Owner

So this is not something I ever want to see here: this is trying to collect payment for the use of the module on behalf of someone else, and promoting the 'tea' and 'pkgx' systems that appear to be throwing as much LLM as they can at the thing.

There's more discussion here, and it's welcome on this thread if you're not on Mastodon.

At the moment my stance is that 'tea is considered harmful'. Given the LLM-generated descriptions on pkgx, and the stable diffusion-ish images there, I'm inclined to think the same about that project.

@xvello
Copy link

xvello commented Feb 27, 2024

@mxcl @ryanml @michaelessiet this is obvious proof that the person behind the '0x9BAac40A197bbb110ae5B2f31342D8d15D734832' wallet is paying people to steal other project's ownership. What will you do to punish that person and all the others who are running the same scam and wasting maintainer time? There's a nice list in the PRs linking back to this one.

@mxcl
Copy link

mxcl commented Feb 27, 2024

Hey, we will be punishing this user, probably with a full ban and tracing as much as possible all activity they may have with other accounts and banning those too. If I could I would also ban all their friends and family too. This behavior is disgusting and counter productive to our mission. I am furious about it.

Additionally we are taking steps to force users to prove they can commit before allowing them to generate the YAML which should prevent idiots like @onedionys from being able to figure out how to create the file.

tea aims to change the nature of open source sustainability. Our mission is pure and we want to help projects like yours to flourish.

pkgx was using LLMs to generate some images and descriptions but we have removed them due to concern from the community. Nothing else is using AI or LLMs in pkgx or tea.

@onedionys
Copy link
Author

Hey, we will be punishing this user, probably with a full ban and tracing as much as possible all activity they may have with other accounts and banning those too. If I could I would also ban all their friends and family too. This behavior is disgusting and counter productive to our mission. I am furious about it.

Additionally we are taking steps to force users to prove they can commit before allowing them to generate the YAML which should prevent idiots like @onedionys from being able to figure out how to create the file.

tea aims to change the nature of open source sustainability. Our mission is pure and we want to help projects like yours to flourish.

pkgx was using LLMs to generate some images and descriptions but we have removed them due to concern from the community. Nothing else is using AI or LLMs in pkgx or tea.

I'm sorry, I just joined the crypto/airdrop community not even a week ago. And for what I did, I just followed someone doing the same thing and it worked, so I followed that method by doing PR to someone's repo.

I didn't know that it wasn't allowed, because in the article I read, I had to contribute to someone's repo and add tea.yml OR add it to my own repo. I admit my mistake and will not make this mistake again and make it a lesson for the future, sir.

I apologize @aredridel for trying to do a Pull Request and wasting your precious time.

@xvello
Copy link

xvello commented Feb 27, 2024

which should prevent idiots like @onedionys from being able to figure out how to create the file.

Wow, slow down @mxcl. Calling people names is not constructive not warranted here. If someone were to deserve such an adjective, it would be the person who designs a system that financially incentivizes malicious actors without implementing proper guardrails against such actors. And no, shifting the moderation burden to Open-Source maintainers is not a proper guardrail.

@mxcl
Copy link

mxcl commented Feb 27, 2024

I apologize @onedionys and @xvello, there is no call for my behavior.

Our guardrail will be making it so onboarding a project requires a check against that project via the GitHub API, it should prevent these types of pull requests and reduce the burden to zero on open source maintainers.

Our goal is to make open source sustainable and we failed here, but it will be fixed by end of day.

@aredridel
Copy link
Owner

I have some really deep doubts that's true: the incentives you're creating here are pretty obviously easy to abuse.

This was referenced Feb 27, 2024
@aredridel aredridel mentioned this pull request Feb 27, 2024
Closed
@rugk
Copy link

rugk commented Feb 27, 2024

Maybe we should write a bot that finds these PRs on GitHub and uses an LLM to comment on all the PRs to mark them as spam and additionally reports the user accounts to GitHub? 🙃

@xvello
Copy link

xvello commented Feb 28, 2024

@mxcl the PR spam is continuing, see github search:

@oldherl oldherl mentioned this pull request Feb 29, 2024
@stuaxo
Copy link

stuaxo commented Mar 2, 2024

Maybe we should write a bot that finds these PRs on GitHub and uses an LLM to comment on all the PRs to mark them as spam and additionally reports the user accounts to GitHub? 🙃

I'm not sure you even need an LLM if all the comments are the same.

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.

None yet

8 participants