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

ER-Diagrams in data_schemas.md #367

Merged
merged 2 commits into from
Jan 5, 2023
Merged

Conversation

jojopirker
Copy link
Contributor

I wanted to better understand the databases involved so I made some ER-diagrams. May be helpful for other people as well so I though I might share it. I necessary I can also share to raw xml files (from draw.io).

@jojopirker jojopirker marked this pull request as ready for review January 4, 2023 13:57
@jojopirker jojopirker mentioned this pull request Jan 4, 2023
Copy link
Collaborator

@fozziethebeat fozziethebeat left a comment

Choose a reason for hiding this comment

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

I love documentation~

@andreaskoepf
Copy link
Collaborator

Thanks a lot for the documentation/diagram! I am a bit worried that it will be outdated during the next days since there will be a couple of changes. But I guess that is always the problem with documentation. @yk should we merge it anyway?

@jojopirker
Copy link
Contributor Author

We can also wait a few days, and I'll update it then.

Copy link
Collaborator

@andreaskoepf andreaskoepf left a comment

Choose a reason for hiding this comment

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

In a future update you could mention the sqllite db of the discrod-bot. Schema is still microscopic .. right now, will probably grow a bit over the next days.. https://github.com/LAION-AI/Open-Assistant/blob/main/discord-bot/bot/db/schema.sql


## Databases

Open-Assistant uses two databases, one for the backend and one for the frontend.
Copy link
Collaborator

Choose a reason for hiding this comment

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

there is actually a third one for the bot which is a small sqlite db. So backend and frontends all have their own persistent state.

@andreaskoepf
Copy link
Collaborator

We can also wait a few days, and I'll update it then.

I'll merge it now. Woulb be really cool if you check in 1-2 weeks and update it to the latest state! :-) Thanks for your contribution!

@andreaskoepf andreaskoepf merged commit 8942194 into LAION-AI:main Jan 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants