Merge pull request #968 from Carlos-err406/fix/double-loop

fix: unnecesary double loop

Merge pull request #968 from Carlos-err406/fix/double-loop
fix: unnecesary double loop
19d49df7 · Timothy Jaeryang Baek · GitHub · 941ab5d6 · 30c94ff1 · 19d49df7
Unverified Commit 19d49df7 authored Feb 29, 2024 by Timothy Jaeryang Baek Committed by GitHub Feb 29, 2024
Show whitespace changes
Inline Side-by-side

Showing with 6 additions and 4 deletions

src/lib/components/chat/Settings/Interface.svelte src/lib/components/chat/Settings/Interface.svelte +6 -4

No files found.
--- a/src/lib/components/chat/Settings/Interface.svelte
+++ b/src/lib/components/chat/Settings/Interface.svelte
@@ -191,10 +191,12 @@
 						placeholder="Select a model"
 					>
 						<option value="" selected>Current Model</option>
-						{#each $models.filter((m) => m.size != null) as model}
-							<option value={model.name} class="bg-gray-100 dark:bg-gray-700"
-								>{model.name + ' (' + (model.size / 1024 ** 3).toFixed(1) + ' GB)'}</option
-							>
+						{#each $models as model}
+							{#if model.size != null}
+								<option value={model.name} class="bg-gray-100 dark:bg-gray-700">
+									{model.name + ' (' + (model.size / 1024 ** 3).toFixed(1) + ' GB)'}
+								</option>
+							{/if}
 						{/each}
 					</select>
 				</div>