fix(server): fix setPrimarySelection bug and replace unreliable trivia API

- Use view store's setDataForAllViews instead of non-existent
  setPrimarySelection on dataset store
- Replace numbersapi.com (unreachable) with catfact.ninja API

Author: PaulHax

server/examples/example_api.py

@@ -21,13 +21,14 @@ def add(a: int, b: int):
     return a + b
 
 
-@volview.expose  # exposes as "number_trivia"
-@volview.expose("get_number_trivia")  # exposes as "get_number_trivia"
-async def number_trivia():
+@volview.expose  # exposes as "cat_fact"
+@volview.expose("get_cat_fact")  # exposes as "get_cat_fact"
+async def cat_fact():
     async with aiohttp.ClientSession() as session:
-        url = "http://numbersapi.com/random/"
+        url = "https://catfact.ninja/fact"
         async with session.get(url) as resp:
-            return await resp.text()
+            data = await resp.json()
+            return data["fact"]
 
 
 @volview.expose("progress")
@@ -83,8 +84,8 @@ def get_base_image(state: ClientState, img_id: str) -> str:
 
 
 async def show_image(img_id: str):
-    store = get_current_client_store("dataset")
-    await store.setPrimarySelection(img_id)
+    store = get_current_client_store("view")
+    await store.setDataForAllViews(img_id)
 
 
 @volview.expose("medianFilter")

server/examples/example_class_api.py

@@ -49,8 +49,8 @@ class ExampleApi:
     def __init__(self):
         self.rpc_router = RpcRouter()
         self.rpc_router.add_endpoint("add", self.add)
-        self.rpc_router.add_endpoint("number_trivia", self.number_trivia)
-        self.rpc_router.add_endpoint("get_number_trivia", self.number_trivia)
+        self.rpc_router.add_endpoint("cat_fact", self.cat_fact)
+        self.rpc_router.add_endpoint("get_cat_fact", self.cat_fact)
         self.rpc_router.add_endpoint("progress", self.number_stream)
         self.rpc_router.add_endpoint("medianFilter", self.median_filter)
 
@@ -61,13 +61,14 @@ def __init__(self):
     def add(self, a, b):
         return a + b
 
-    ## number trivia example ##
+    ## cat fact example ##
 
-    async def number_trivia(self):
+    async def cat_fact(self):
         async with aiohttp.ClientSession() as session:
-            url = "http://numbersapi.com/random/"
+            url = "https://catfact.ninja/fact"
             async with session.get(url) as resp:
-                return await resp.text()
+                data = await resp.json()
+                return data["fact"]
 
     ## progress bar example ##
 
@@ -121,8 +122,8 @@ def _get_base_image(self, img_id):
         return img_id
 
     async def _show_image(self, img_id):
-        store = get_current_client_store("dataset")
-        await store.setPrimarySelection(img_id)
+        store = get_current_client_store("view")
+        await store.setDataForAllViews(img_id)
 
 
 volview = VolViewApi()

src/components/ServerModule.vue

@@ -25,17 +25,17 @@ const doSum = async () => {
   }
 };
 
-// --- number trivia --- //
+// --- cat fact --- //
 
-const trivia = ref('');
-const triviaLoading = ref(false);
+const catFact = ref('');
+const catFactLoading = ref(false);
 
-const getTrivia = async () => {
-  triviaLoading.value = true;
+const getCatFact = async () => {
+  catFactLoading.value = true;
   try {
-    trivia.value = (await client.call('number_trivia')) as string;
+    catFact.value = (await client.call('cat_fact')) as string;
   } finally {
-    triviaLoading.value = false;
+    catFactLoading.value = false;
   }
 };
 
@@ -122,21 +122,21 @@ const hasCurrentImage = computed(() => !!currentImageID.value);
       </v-col>
     </v-row>
     <v-divider />
-    <v-list-subheader>Trivia</v-list-subheader>
+    <v-list-subheader>Cat Fact</v-list-subheader>
     <v-row>
-      <v-btn @click="getTrivia" :loading="triviaLoading" :disabled="!ready">
-        Random number trivia
+      <v-btn @click="getCatFact" :loading="catFactLoading" :disabled="!ready">
+        Random cat fact
       </v-btn>
     </v-row>
     <v-row>
       <v-col>
-        <label for="remote-trivia-text">
+        <label for="remote-cat-fact-text">
           <textarea
-            id="remote-trivia-text"
+            id="remote-cat-fact-text"
             readonly
             class="text-white"
             style="width: 100%"
-            :value="trivia"
+            :value="catFact"
           />
         </label>
       </v-col>