Skip to content

Commit 09e4c0a

Browse files
authored
fix:move code to dependencies (#571)
* fox:update requirements update imports and requirements for latest pipeline plugins signature * remove methods marked for deprecation * fixes * fix re-activation of skills that just deactivated themselves during the match * account for session_id * remove deleted messages from tests * remove deleted messages from tests * update tests * fix converse * update tests * @coderabbitai suggestion * revert change, we still need to update timestamps on active skills * fix get_response * fix:workshop2_compat * fix:workshop2_compat * fix:workshop2_compat * fix:workshop2_compat * fix:workshop2_compat * fix:workshop2_compat * fix:workshop2_compat * fix:workshop2_compat * fix:workshop2_compat * fix:workshop2_compat * fix:workshop2_compat * deprecation logs * deprecation logs * tests moved to adapt repo * fix deprecation warning version * fix adapt init * keep intent api unittest * dont access deprecated stuff in tests
1 parent 4c7cbaa commit 09e4c0a

34 files changed

+678
-1297
lines changed

mycroft/skills/common_play_skill.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -158,17 +158,14 @@ def __handle_play_start(self, message):
158158
data = message.data.get("callback_data")
159159

160160
# Stop any currently playing audio
161-
if self.audioservice and self.audioservice.is_playing:
162-
self.audioservice.stop()
161+
self.stop()
162+
163163
message.context["skill_id"] = self.skill_id
164-
self.bus.emit(message.forward("mycroft.stop"))
165164

166165
# Save for CPS_play() later, e.g. if phrase includes modifiers like
167166
# "... on the chromecast"
168167
self.play_service_string = phrase
169168

170-
self.activate()
171-
172169
# Invoke derived class to provide playback data
173170
self.CPS_start(phrase, data)
174171

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
1-
from ovos_core.intent_services import AdaptService,\
2-
ConverseService, \
3-
CommonQAService, \
4-
FallbackService, \
5-
PadaciosoService
6-
from ovos_core.intent_services import IntentMatch
7-
from mycroft.skills.intent_services.adapt_service import AdaptIntent, IntentBuilder, Intent
1+
from ovos_core.intent_services.fallback_service import FallbackService
2+
from ovos_core.intent_services.converse_service import ConverseService
3+
from ovos_adapt.opm import AdaptPipeline as AdaptService
4+
from padacioso.opm import PadaciosoPipeline as PadaciosoService
5+
from ovos_commonqa.opm import CommonQAService
6+
from ovos_plugin_manager.templates.pipeline import IntentMatch
7+
from ovos_workshop.intents import Intent as AdaptIntent, IntentBuilder, Intent
88

99
try:
10-
from ovos_core.intent_services.padatious_service import PadatiousService, PadatiousMatcher
10+
from ovos_padatious.opm import PadatiousPipeline as PadatiousService, PadatiousMatcher
1111
except ImportError:
1212
from ovos_utils.log import LOG
1313
LOG.warning("padatious not installed")
14-
from ovos_core.intent_services.padacioso_service import PadaciosoService as PadatiousService
14+
from padacioso.opm import PadaciosoPipeline as PadatiousService

mycroft/skills/intent_services/adapt_service.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,11 @@
1313
# limitations under the License.
1414
#
1515
"""An intent parsing service using the Adapt parser."""
16-
from ovos_adapt.context import ContextManagerFrame
1716
from ovos_adapt.engine import IntentDeterminationEngine
1817
from ovos_workshop.intents import IntentBuilder, Intent
19-
from ovos_adapt.opm import ContextManager, AdaptPipeline as AdaptService
18+
from ovos_adapt.opm import AdaptPipeline as AdaptService
19+
from ovos_bus_client.session import IntentContextManagerFrame as ContextManagerFrame, \
20+
IntentContextManager as ContextManager
2021

2122

2223
class AdaptIntent(IntentBuilder):
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
from ovos_core.intent_services.commonqa_service import CommonQAService
1+
from ovos_commonqa.opm import CommonQAService
22
EXTENSION_TIME = 10

mycroft/skills/intent_services/padatious_service.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,5 +13,5 @@
1313
# limitations under the License.
1414
#
1515
"""Intent service wrapping padatious."""
16-
from ovos_core.intent_services.padatious_service import PadatiousMatcher, PadatiousService, PadatiousIntent
16+
from ovos_padatious.opm import PadatiousMatcher, PadatiousPipeline as PadatiousService, PadatiousIntent
1717

0 commit comments

Comments
 (0)