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 · 4 comments · May be fixed by #538
Open

[Feature] Rating system for restaurants #360

cirex-web opened this issue Dec 14, 2023 · 4 comments · May be fixed by #538
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).

@cirex-web
Copy link
Contributor Author

SQL sounds good

@T7alabdullah
Copy link

Hello! I’m a student at Carnegie Mellon University in Qatar, and my team and I are working on a Software Engineering project for our course. We’re required to contribute to open-source projects, and this issue caught our attention.. it looks like a great fit for what we’re learning right now.
Would it be alright if we took this on as part of our project? We’ll make sure to follow the contribution guidelines and do our best to add value to the project

@mthani2 mthani2 linked a pull request Dec 11, 2024 that will close this issue
8 tasks
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

Successfully merging a pull request may close this issue.

3 participants