Merge pull request #62 from OpenVoiceOS/release-0.3.0a2

Release 0.3.0a2

CHANGELOG.md

@@ -1,12 +1,21 @@
 # Changelog
 
-## [0.2.15a1](https://github.com/OpenVoiceOS/ovos-skill-wolfie/tree/0.2.15a1) (2024-12-05)
+## [0.3.0a2](https://github.com/OpenVoiceOS/ovos-skill-wolfie/tree/0.3.0a2) (2025-01-04)
 
-[Full Changelog](https://github.com/OpenVoiceOS/ovos-skill-wolfie/compare/0.2.14...0.2.15a1)
+[Full Changelog](https://github.com/OpenVoiceOS/ovos-skill-wolfie/compare/0.2.16a1...0.3.0a2)
 
 **Merged pull requests:**
 
-- fix: typo in catalan \(invalid syntax\) [\#57](https://github.com/OpenVoiceOS/ovos-skill-wolfie/pull/57) ([JarbasAl](https://github.com/JarbasAl))
+- Update version.py [\#61](https://github.com/OpenVoiceOS/ovos-skill-wolfie/pull/61) ([JarbasAl](https://github.com/JarbasAl))
+- refactor: new common query decorators [\#60](https://github.com/OpenVoiceOS/ovos-skill-wolfie/pull/60) ([JarbasAl](https://github.com/JarbasAl))
+
+## [0.2.16a1](https://github.com/OpenVoiceOS/ovos-skill-wolfie/tree/0.2.16a1) (2024-12-12)
+
+[Full Changelog](https://github.com/OpenVoiceOS/ovos-skill-wolfie/compare/0.2.15...0.2.16a1)
+
+**Merged pull requests:**
+
+- adjusting German translation from joergz2 [\#59](https://github.com/OpenVoiceOS/ovos-skill-wolfie/pull/59) ([gitlocalize-app[bot]](https://github.com/apps/gitlocalize-app))
 
 
 

__init__.py

@@ -13,30 +13,87 @@
 
 import tempfile
 from os.path import join, isfile
-from typing import Optional
+from typing import Optional, Tuple
 
 import requests
-from ovos_backend_client.api import WolframAlphaApi as _WA
 from ovos_bus_client import Message
 from ovos_bus_client.session import SessionManager
 from ovos_config import Configuration
 from ovos_plugin_manager.templates.solvers import QuestionSolver
 from ovos_utils import classproperty
 from ovos_utils.process_utils import RuntimeRequirements
-from ovos_workshop.decorators import intent_handler
-from ovos_workshop.skills.common_query_skill import CommonQuerySkill, CQSMatchLevel
+from ovos_workshop.decorators import intent_handler, common_query
+from ovos_workshop.skills.ovos import OVOSSkill
 
 
-class WolframAlphaApi(_WA):
+class WolframAlphaApi:
+    def __init__(self, key: str):
+        self.key = key or "Y7R353-9HQAAL8KKA"
+
+    @staticmethod
+    def _get_lat_lon(**kwargs):
+        lat = kwargs.get("latitude") or kwargs.get("lat")
+        lon = kwargs.get("longitude") or kwargs.get("lon") or kwargs.get("lng")
+        if not lat or not lon:
+            cfg = Configuration().get("location", {}).get("coordinate", {})
+            lat = cfg.get("latitude")
+            lon = cfg.get("longitude")
+        return lat, lon
+
+    def spoken(self, query, units="metric", lat_lon=None, optional_params=None):
+        optional_params = optional_params or {}
+        if not lat_lon:
+            lat_lon = self._get_lat_lon(**optional_params)
+        params = {'i': query,
+                  "geolocation": "{},{}".format(*lat_lon),
+                  'units': units,
+                  "appid": self.key,
+                  **optional_params}
+        url = 'https://api.wolframalpha.com/v1/spoken'
+        return requests.get(url, params=params).text
+
+    def simple(self, query, units="metric", lat_lon=None, optional_params=None):
+        optional_params = optional_params or {}
+        if not lat_lon:
+            lat_lon = self._get_lat_lon(**optional_params)
+        params = {'i': query,
+                  "geolocation": "{},{}".format(*lat_lon),
+                  'units': units,
+                  "appid": self.key,
+                  **optional_params}
+        url = 'https://api.wolframalpha.com/v1/simple'
+        return requests.get(url, params=params).text
+
+    def full_results(self, query, units="metric", lat_lon=None, optional_params=None):
+        """Wrapper for the WolframAlpha Full Results v2 API.
+        https://products.wolframalpha.com/api/documentation/
+        Pods of interest
+        - Input interpretation - Wolfram's determination of what is being asked about.
+        - Name - primary name of
+        """
+        optional_params = optional_params or {}
+        if not lat_lon:
+            lat_lon = self._get_lat_lon(**optional_params)
+        params = {'input': query,
+                  "units": units,
+                  "mode": "Default",
+                  "format": "image,plaintext",
+                  "geolocation": "{},{}".format(*lat_lon),
+                  "output": "json",
+                  "appid": self.key,
+                  **optional_params}
+        url = 'https://api.wolframalpha.com/v2/query'
+        data = requests.get(url, params=params)
+        return data.json()
+
     def get_image(self, query: str, units: Optional[str] = None):
         """
         query assured to be in self.default_lang
         return path/url to a single image to acompany spoken_answer
         """
-        # TODO - extend backend-client method for picture
         units = units or Configuration().get("system_unit", "metric")
         url = 'http://api.wolframalpha.com/v1/simple'
-        params = {"appid": self.credentials["wolfram"],
+        params = {"appid": self.key,
                   "i": query,
                   # "background": "F5F5F5",
                   "layout": "labelbar",
@@ -59,8 +116,6 @@ def __init__(self, config=None):
         config["lang"] = "en"  # only supports english
         super().__init__(config=config)
         self.api = WolframAlphaApi(key=self.config.get("appid") or "Y7R353-9HQAAL8KKA")
-        # TODO - debug, key doesnt seem to be passed along to base class ???
-        self.api.backend.credentials = self.api.credentials
 
     @staticmethod
     def make_speakable(summary: str):
@@ -216,7 +271,7 @@ def get_expanded_answer(self, query,
         return [s for s in steps if s]
 
 
-class WolframAlphaSkill(CommonQuerySkill):
+class WolframAlphaSkill(OVOSSkill):
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self.session_results = {}  # session_id: {}
@@ -255,7 +310,16 @@ def handle_search(self, message: Message):
             self.speak_dialog("no_answer")
 
     # common query integration
-    def CQS_match_query_phrase(self, phrase: str):
+    def cq_callback(self, utterance: str, answer: str, lang: str):
+        """ If selected show gui """
+        # generate image for the query after skill was selected for speed
+        image = self.wolfie.visual_answer(utterance, lang=lang, units=self.system_unit)
+        self.gui["wolfram_image"] = image or "logo.png"
+        # scrollable full result page
+        self.gui.show_page("wolf", override_idle=45)
+
+    @common_query(callback=cq_callback)
+    def match_common_query(self, phrase: str, lang: str) -> Tuple[str, float]:
         self.log.debug("WolframAlpha query: " + phrase)
         if self.wolfie is None:
             self.log.error("WolframAlphaSkill not initialized, no response")
@@ -264,24 +328,16 @@ def CQS_match_query_phrase(self, phrase: str):
         sess = SessionManager.get()
         self.session_results[sess.session_id] = {"phrase": phrase,
                                                  "image": None,
-                                                 "lang": sess.lang,
+                                                 "lang": lang,
                                                  "system_unit": sess.system_unit,
                                                  "spoken_answer": None}
 
-        response = self.ask_the_wolf(phrase, sess.lang, sess.system_unit)
+        response = self.ask_the_wolf(phrase, lang, sess.system_unit)
         if response:
             self.session_results[sess.session_id]["spoken_answer"] = response
             self.log.debug(f"WolframAlpha response: {response}")
-            return (phrase, CQSMatchLevel.EXACT, response,
-                    {'query': phrase, 'answer': response})
+            return response, 0.7
 
-    def CQS_action(self, phrase: str, data: dict):
-        """ If selected show gui """
-        # generate image for the query after skill was selected for speed
-        image = self.wolfie.visual_answer(phrase, lang=self.lang, units=self.system_unit)
-        self.gui["wolfram_image"] = image or f"{self.root_dir}/res/logo.png"
-        # scrollable full result page
-        self.gui.show_page("wolf", override_idle=45)
 
     # wolfram integration
     def ask_the_wolf(self, query: str,

locale/de-de/search_wolfie.intent

@@ -1,9 +1,10 @@
-Fragen Sie den Wolf alpha über {query}
+Frage Wolfram Alpha über {query}
 Suche Wolf für {query}
 Suche den Wolf nach {query}
 Suche wolfram alpha nach {query}
-fragen Sie den Wolf {query}
-fragen Sie den Wolf über {query}
-was sagt Wolf alpha über {query}
+befragen den Wolf über {query}
+frage den Wolf nach {query}
+frage den Wolf {query}
+was sagt Wolfram alpha über {query}
 was sagt Wolfram über {query}
 was sagt der Wolf über {query}

requirements.txt

@@ -1,5 +1,5 @@
 ovos-translate-server-plugin
 ovos-config>=0.0.12
 ovos-utils>=0.0.38
-ovos_workshop>=0.0.16,<4.0.0
-ovos-plugin-manager>=0.0.26,<1.0.0
+ovos_workshop>=3.3.0,<4.0.0
+ovos-plugin-manager>=0.0.26,<1.0.0

translations/de-de/intents.json

@@ -1,13 +1,13 @@
 {
-    "search_wolfie.intent": [
-        "fragen Sie den Wolf \u00fcber {query}",
-        "Suche den Wolf nach {query}",
-        "was sagt Wolfram \u00fcber {query}",
-        "was sagt der Wolf \u00fcber {query}",
-        "Suche Wolf f\u00fcr {query}",
-        "was sagt Wolf alpha \u00fcber {query}",
-        "Suche wolfram alpha nach {query}",
-        "fragen Sie den Wolf {query}",
-        "Fragen Sie den Wolf alpha \u00fcber {query}"
-    ]
-}
+  "search_wolfie.intent": [
+    "befragen den Wolf über {query}\nfrage den Wolf nach {query}",
+    "Suche den Wolf nach {query}",
+    "was sagt Wolfram über {query}",
+    "was sagt der Wolf über {query}",
+    "Suche Wolf für {query}",
+    "was sagt Wolfram alpha über {query}",
+    "Suche wolfram alpha nach {query}",
+    "frage den Wolf {query}",
+    "Frage Wolfram Alpha über {query}"
+  ]
+}

version.py

@@ -1,6 +1,6 @@
 # START_VERSION_BLOCK
 VERSION_MAJOR = 0
-VERSION_MINOR = 2
-VERSION_BUILD = 15
-VERSION_ALPHA = 0
+VERSION_MINOR = 3
+VERSION_BUILD = 0
+VERSION_ALPHA = 2
 # END_VERSION_BLOCK