Check if GUI is connected rather than maintain list of platforms

There are already many Mycroft platforms that have GUIs
and this will only grow. We want to know if the device
has a GUI connected rather than if it is in a pre-defined
list of platforms.

Author: krisgesling

mycroft/skills/common_query_skill.py

@@ -36,8 +36,6 @@ def is_CQSVisualMatchLevel(match_level):
     return isinstance(match_level, type(CQSVisualMatchLevel.EXACT))
 
 
-VISUAL_DEVICES = ['mycroft_mark_2']
-
 """these are for the confidence calculation"""
 # how much each topic word is worth
 # when found in the answer
@@ -53,10 +51,6 @@ def is_CQSVisualMatchLevel(match_level):
 WORD_COUNT_DIVISOR = 100
 
 
-def handles_visuals(platform):
-    return platform in VISUAL_DEVICES
-
-
 class CommonQuerySkill(MycroftSkill, ABC):
     """Question answering skills should be based on this class.
 
@@ -149,9 +143,8 @@ def __calc_confidence(self, match, phrase, level, answer):
         num_sentences = float(float(len(answer.split("."))) / float(10))
 
         # Add bonus if match has visuals and the device supports them.
-        platform = self.config_core.get("enclosure", {}).get("platform")
         bonus = 0.0
-        if is_CQSVisualMatchLevel(level) and handles_visuals(platform):
+        if is_CQSVisualMatchLevel(level) and self.gui.connected:
             bonus = 0.1
 
         # extract topic

test/unittests/skills/test_common_query_skill.py

@@ -23,11 +23,6 @@ def test_lifecycle(self):
         bus.on.assert_any_call('question:action', AnyCallable())
         skill.shutdown()
 
-    def test_handles_visuals(self):
-        """Test the helper method to determine if the skill handles visuals."""
-        self.assertTrue(handles_visuals('mycroft_mark_2'))
-        self.assertFalse(handles_visuals('mycroft_mark_1'))
-
     def test_common_test_skill_action(self):
         """Test that the optional action is triggered."""
         query_action = self.bus.on.call_args_list[-1][0][1]