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

[Feature] Rating system for restaurants #360

Open
cirex-web opened this issue Dec 14, 2023 · 2 comments
Open

[Feature] Rating system for restaurants #360

cirex-web opened this issue Dec 14, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@cirex-web
Copy link
Contributor

cirex-web commented Dec 14, 2023

Description

If people are actually consistently using this site, maybe it'll be helpful to see/contribute restaurant ratings and reviews. This will also put CMUeats one level above the normal CMU dining website.

I don't think we have a database yet, but I'm open to using whatever.

Designs

No response

@cirex-web cirex-web added the enhancement New feature or request label Dec 14, 2023
@cirex-web cirex-web changed the title [Feature] Rating system for restaurants? [Feature] Rating system for restaurants Dec 14, 2023
@GhostOf0days
Copy link
Member

Good idea.

@GhostOf0days
Copy link
Member

GhostOf0days commented Jan 18, 2024

Now that we have migrated to Bun, Bun has access to a super fast version of SQLite3. While structured data and SQL database does make more sense, there are some performance optimizations that can be made by using unstructured data due to extra empty space when using structured data (such as in an SQL database). For an example, see fiftyone. Also, see slides. Although their product focuses on machine learning, they were able to achieve performance optimizations through clever techniques. I am down for either way (built-in Bun SQLite database or non-structured approach).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants