search

LAION-AI / Open-Assistant

OpenAssistant is a chat-based assistant that understands tasks, can interact with third-party systems, and retrieve information dynamically to do so.
https://open-assistant.io
Apache License 2.0
36.92k stars 3.22k forks source link

Add paper acknowledgement form #3637

Closed yk closed 1 year ago

yk commented 1 year ago

I know all of this is suboptimal, but I'd like to get people to sign up to be mentioned asap. Feel free to improve (or to tell me how to improve) now or in subsequent PRs. I anticipate this form will be going away in the future, so I'm not sure a lot of work will be really worth it.

AbdBarho commented 1 year ago

Hey sorry I have guests over I will try to give it a look in the evening, what is the context for this? is it required as part of the paper submission?

yk commented 1 year ago

Hey sorry I have guests over I will try to give it a look in the evening, what is the context for this? is it required as part of the paper submission?

It's not required, no. But it would be nice to acknowledge contributors. So it's not a super stress, but we should get it going in the next days.

AbdBarho commented 1 year ago

Would it make sense to allow the users to change their user name, and then make the leaderboard public? with all users with more than 0 points?

yk commented 1 year ago

Would it make sense to allow the users to change their user name, and then make the leaderboard public? with all users with more than 0 points?

we could do that as well, but for a lot of people I think it would be cool to see their name listed in a published paper.

AbdBarho commented 1 year ago

Oh, of course, that was my intention as well, my proposal was to read from the leaderboards, but I guess we would need consent and a "real" name anyway, so no problem.

I took the liberty of cleaning up some stuff so it is not a "hack" anymore, I hope you are fine with that

Although the database changes are not the best, it is not a problem since they should be only temporary.

Please note: we still use npx prisma db push which just overwrites the db schema for our deployment https://github.com/LAION-AI/Open-Assistant/blob/0d4adb5f1ad6c38a828370414a584dd485165dce/website/wait-for-postgres.sh#L27 So adding columns is okay, migrations still not there yet, even though I added something in #3488, it was never merged.