add average rating as a column

rommapp · Jan 10, 2025 · b5d2cad · b5d2cad
1 parent 9e4e716
commit b5d2cad
Show file tree

Hide file tree

Showing 6 changed files with 56 additions and 13 deletions.
diff --git a/backend/endpoints/responses/rom.py b/backend/endpoints/responses/rom.py
@@ -121,6 +121,7 @@ class RomSchema(BaseModel):
     # Metadata fields
     first_release_date: int | None
     youtube_video_id: str | None
+    average_rating: float
     alternative_names: list[str]
     genres: list[str]
     franchises: list[str]

diff --git a/backend/models/rom.py b/backend/models/rom.py
@@ -170,6 +170,22 @@ def first_release_date(self) -> int:
             return self.igdb_metadata.get("first_release_date", 0) * 1000
         return 0
 
+    @property
+    def average_rating(self) -> float:
+        igdb_rating = (
+            float(self.igdb_metadata.get("total_rating", 0))
+            if self.igdb_metadata
+            else 0.0
+        )
+        moby_rating = (
+            float(self.moby_metadata.get("moby_score", 0))
+            if self.moby_metadata
+            else 0.0
+        )
+        ratings = [igdb_rating, moby_rating * 10]  # Moby rating is out of 10
+
+        return sum(ratings) / len([r for r in ratings if r]) if any(ratings) else 0.0
+
     @property
     def genres(self) -> list[str]:
         return (

diff --git a/frontend/src/__generated__/models/DetailedRomSchema.ts b/frontend/src/__generated__/models/DetailedRomSchema.ts
diff --git a/frontend/src/__generated__/models/RomSchema.ts b/frontend/src/__generated__/models/RomSchema.ts
diff --git a/frontend/src/__generated__/models/SimpleRomSchema.ts b/frontend/src/__generated__/models/SimpleRomSchema.ts
diff --git a/frontend/src/components/common/Game/Table.vue b/frontend/src/components/common/Game/Table.vue
@@ -64,6 +64,12 @@ const HEADERS = [
     sortable: true,
     key: "first_release_date",
   },
+  {
+    title: "Rating",
+    align: "start",
+    sortable: true,
+    key: "average_rating",
+  },
   {
     title: "Languages",
     align: "start",
@@ -76,7 +82,12 @@ const HEADERS = [
     sortable: false,
     key: "regions",
   },
-  { title: "", align: "center", key: "actions", sortable: false },
+  {
+    title: "",
+    align: "center",
+    key: "actions",
+    sortable: false,
+  },
 ] as const;
 
 const selectedRomIDs = computed(() => selectedRoms.value.map((rom) => rom.id));
@@ -199,30 +210,38 @@ onMounted(() => {
       </td>
     </template>
     <template #item.file_size_bytes="{ item }">
-      <v-chip size="x-small" label>{{
-        formatBytes(item.file_size_bytes)
-      }}</v-chip>
+      <span class="text-no-wrap">{{ formatBytes(item.file_size_bytes) }}</span>
     </template>
     <template #item.created_at="{ item }">
-      <v-chip v-if="item.created_at" size="x-small" label>{{
+      <span v-if="item.created_at" class="text-no-wrap">{{
         new Date(item.created_at).toLocaleDateString("en-US", {
           day: "2-digit",
           month: "short",
           year: "numeric",
         })
-      }}</v-chip>
+      }}</span>
+      <span v-else>-</span>
     </template>
     <template #item.first_release_date="{ item }">
-      <v-chip v-if="item.first_release_date" size="x-small" label>{{
+      <span v-if="item.first_release_date" class="text-no-wrap">{{
         new Date(item.first_release_date).toLocaleDateString("en-US", {
           day: "2-digit",
           month: "short",
           year: "numeric",
         })
-      }}</v-chip>
+      }}</span>
+      <span v-else>-</span>
+    </template>
+    <template #item.average_rating="{ item }">
+      <span v-if="item.average_rating" class="text-no-wrap">{{
+        Intl.NumberFormat("en-US", {
+          maximumSignificantDigits: 3,
+        }).format(item.average_rating)
+      }}</span>
+      <span v-else>-</span>
     </template>
     <template #item.languages="{ item }">
-      <div class="d-flex">
+      <div class="text-no-wrap" v-if="item.languages.length > 0">
         <span
           class="emoji"
           v-for="language in item.languages.slice(0, 3)"
@@ -239,9 +258,10 @@ onMounted(() => {
           }}
         </spa>
       </div>
+      <span v-else>-</span>
     </template>
     <template #item.regions="{ item }">
-      <div class="d-flex">
+      <div class="text-no-wrap" v-if="item.regions.length > 0">
         <span
           class="emoji"
           v-for="region in item.regions.slice(0, 3)"
@@ -250,10 +270,13 @@ onMounted(() => {
         >
           {{ regionToEmoji(region) }}
         </span>
+        <spa class="reglang-super">
+          {{
+            item.regions.length > 3 ? `&nbsp;+${item.regions.length - 3}` : ""
+          }}
+        </spa>
       </div>
-      <spa class="reglang-super">
-        {{ item.regions.length > 3 ? `&nbsp;+${item.regions.length - 3}` : "" }}
-      </spa>
+      <span v-else>-</span>
     </template>
     <template #item.actions="{ item }">
       <v-btn-group density="compact">