Commit Graph

4976 Commits (mark-ii/latest)

Author SHA1 Message Date
jarbasal 87d22d30a4 fix cyclic imports with LOG
The mycroft.util.log module called LOG.init() which needs to read the config,
the configuration module imports the log system to log causing cyclic import
issues.

This moves the LOG.init() call out of the log module making it possible
to use the normal config system to init the logs and does a slight
re-arrangement so that the uninited log can be used.
2021-08-05 10:04:45 +02:00
Åke Forslund 3fd96cf71b WIP Review comments
- Add TODO for 22.02 to remove the compatibility code
- Make Warning a single Log statement
- mycroft-config script now uses XDG-environment variable
- Remove redundant code
- Replace hard coded references to ~/.config
- Explicitly remove new path before move of "filesystem" (if needed)
2021-08-05 10:04:45 +02:00
Åke Forslund 9029dc1f41 Slight cleanup
- Make XDG usage more visible by using the xdg module prefix
- fix overloaded python keywords
- remove unused imports
2021-08-05 10:04:45 +02:00
Åke Forslund 8e69d4616d Correct resolution order for resolving log configs 2021-08-05 10:04:45 +02:00
Åke Forslund 28017993c5 Restore system locations to mycroft.configuration
Fixes issue with Timer skill
2021-08-05 10:04:45 +02:00
Åke Forslund faf101bfcd Increase Jenkins timeout 2021-08-05 10:04:45 +02:00
Bart Ribbers e20443b824 Use XDG Base directories for settings, cache and runtime data
Improve deprecation warning message
2021-08-05 10:04:45 +02:00
Åke Forslund ff9f8e898d Update identity location for VK test
Moved from .mycroft to XDG folder
2021-08-05 10:04:45 +02:00
Kris Gesling 170ebc44ec
Merge pull request #2951 from forslund/feature/then-wait-with-event-handler
Use event handler to capture messages in then_wait()
2021-08-05 15:40:04 +09:30
Kris Gesling e40530a8ac
Merge pull request #2948 from forslund/bugfix/vk-message-race
Fix Possible vk message race
2021-08-05 15:39:12 +09:30
Kris Gesling 313f4e8759
Merge pull request #2946 from MycroftAI/refactor/vk-test-helpers
Improve the speed of waiting for dialogs helper function
2021-08-05 09:29:17 +09:30
Kris Gesling c996008478
Merge pull request #2944 from MycroftAI/bugfix/ci-unique-temp-paths
Use unique paths for temp file storage
2021-08-04 12:05:13 +09:30
Kris Gesling 7c413de32c
Merge pull request #2956 from AIIX/web_enable_request_feature
Add feature request functionality to webviews
2021-08-04 11:42:45 +09:30
Kris Gesling c21e74d4d4
Merge pull request #2960 from MycroftAI/feature/upgrade-adapt
Upgrade Adapt to v0.5.1
2021-08-03 15:25:57 +09:30
Kris Gesling 480c604dfa
Merge pull request #2962 from MycroftAI/feature/configurable-network-tests
Make network tests configurable
2021-08-03 07:55:47 +09:30
Kris Gesling a794db0c9a Make network tests configurable
Previously test URIs were hardcoded.
They can now be configured in mycroft.conf
2021-08-02 21:00:05 +09:30
Kris Gesling b7a0853f3c Upgrade Adapt to v0.5.1
- Fix removal of regex entities
- Update trie dosctrings
- Guarantee sorted results from IntentDeterminationEngine
- Enumerate all possible parse results if context or regex entities are in
play.
- Explicit test to assert results are sorted
- Fix name of LICENSE file in Adapt package
2021-07-27 23:06:43 +09:30
jarbasal 88818bd52f Fix loading of audioservice plugins 2021-07-26 21:17:17 +09:30
Kris Gesling 450a092887
Merge pull request #2957 from forslund/bugfix/load-audioservice-plugin
Fix loading of audioservice plugins
2021-07-26 20:51:47 +09:30
Åke Forslund b40fcf0e93 Update VK then step for checking messagetype
Simplify the function and use the standard then_wait() for the heavy lifting,
this makes it utilize the new event driven functionality.
2021-07-23 09:46:07 +02:00
Åke Forslund 171d3840f3 Remove sleeps from end of scenarios and features 2021-07-23 09:45:44 +02:00
Åke Forslund 578a3ec439 Use event to capture messages in then_wait()
Instead of busily polling the bus for new messages an event handler is
registered (and teared down after check is complete)
2021-07-23 09:45:44 +02:00
jarbasal 24c4ba4a05 Fix loading of audioservice plugins 2021-07-23 09:17:58 +02:00
Aditya Mehra 45d5d9e478 Add feature request functionality to webviews 2021-07-23 15:14:06 +09:30
Chris Veilleux 1d03eadb59
Merge pull request #2954 from forslund/bugfix/vk-missing-stock-skill
VK: Remove mycroft-stock from install list
2021-07-22 14:32:15 -05:00
Åke Forslund cbd17a8dc3 Remove mycroft-stock from install list 2021-07-22 17:20:04 +02:00
Kris Gesling d1816f5933
Merge pull request #2807 from forslund/test/jenkins-update-comment
Add update comment logic to Jenkins
2021-07-14 09:43:49 +09:30
Kris Gesling 31e1e2d857
Merge pull request #2927 from ChanceNCounter/fix/lang-none
stop passing lang=None to Lingua Franca
2021-07-12 21:46:17 +09:30
Chris Veilleux 8f41d176d0 remove print statement used for testing 2021-07-09 16:39:49 -05:00
Chris Veilleux 6fd97b4e85 fixed PEP8 issue 2021-07-09 15:45:07 -05:00
Chris Veilleux fbb1d06adc Merge remote-tracking branch 'origin/refactor/vk-test-helpers' into refactor/vk-test-helpers 2021-07-09 15:43:25 -05:00
Åke Forslund 4b66fb1dd1 Add docstrings to InterceptAllBusClient 2021-07-09 17:06:33 +02:00
Åke Forslund cf355360ad Fix message race when using clear_messages()
This handles a scenario that a message arrives between a call to get_messages() and clear_messages(). clear_messages() will only clear the messages that has been evaluated atleast once.

A new method clear_all_messages() has been added to clear the entire message stack and is used between scenarios to reset the list.
2021-07-09 17:06:30 +02:00
Kris Gesling b3a0b3b1e4
Merge pull request #2938 from forslund/bugfix/tts-returned-cache-path
Fix TTS using the returned path
2021-07-08 11:56:32 +09:30
Chris Veilleux 10d1dcfe04
Merge pull request #2945 from MycroftAI/refactor/race-condition-comment
Added comments to document race condition
2021-07-07 02:17:59 -05:00
Chris Veilleux 105a5b4be4 Improve the speed of waiting for dialogs by exiting the loop after a match is spoken. Also provide error handling for when a match is not found. 2021-07-06 14:26:23 -05:00
Chris Veilleux 18cb28088d Added comments to document race condition 2021-07-06 13:48:52 -05:00
Kris Gesling 2deab67021
Merge pull request #2943 from MycroftAI/feature/localization-fa-ir
fa-ir localization initialized
2021-07-06 22:16:44 +09:30
Kris Gesling d14e6b65d7 Use unique paths for temp file storage
When transferring the Allure report and the Mycroft logs to the
report host, the zip files were being written to a common
directory. In the event that multiple jobs were writing to or
reading from this directory at the same time, conflicts could
occur.

This ensures that both zip files are written to unique paths
and cleaned up afterward.
2021-07-06 12:41:35 +09:30
HKalbasi 28c52fef75 fa-ir localization initialized (#2778)
==== Localization Notes ====
fa-ir: 35 strings added
2021-07-06 07:10:23 +09:30
Kris Gesling 9b188595da
Merge pull request #2939 from MycroftAI/fix_bargein
Add TTS pause and resume to fix barge in
2021-07-03 09:15:04 +09:30
Chris Veilleux 80f9eb053c
Merge pull request #2937 from MycroftAI/bugfix/vk-race-condition
Fix VK race condition in MycroftSkill.get_response()
2021-07-02 15:03:03 -05:00
Ken Smith ff01b7388b add pause and resume ability to tts playback thread and use it during barge in 2021-07-02 12:12:07 -04:00
Åke Forslund 0ae02b5939 Fix TTS using the returned path 2021-07-02 08:34:03 +02:00
Chris Veilleux 3959267726 Reordered wait_while_speaking and sleep to fix a race condition that was occurring with the converse logic in MycroftSkill.get_response() 2021-07-01 22:31:22 -05:00
Kris Gesling 6099da18b3
Merge pull request #2934 from MycroftAI/bugfix/tts-audio-file-path
Fix #2929: use returned TTS audio file path
2021-06-30 07:03:01 +09:30
Kris Gesling 0394568704 Use returned TTS audio file path
This reverts an unintentional breaking change.
A TTS engine may return a different file path than was requested.
This again uses the returned path but adds a deprecation warning
that this behaviour will no longer be supported in an upcoming release.

Fixes #2929
2021-06-29 21:53:53 +09:30
Kris Gesling 0cbd040a55
Merge pull request #2900 from MycroftAI/bugfix/common-play-active
Add Common Play to active Skills when invoked
2021-06-29 20:59:33 +09:30
Kris Gesling 27165148d4
Merge pull request #2936 from MycroftAI/bugfix/readme-skill-config
Remove docs encouraging Skill config via mycroft.conf
2021-06-29 09:02:37 +09:30
Kris Gesling cfe03a4285 Remove docs encouraging Skill config via mycroft.conf 2021-06-28 16:55:03 +09:30