Commit Graph

3843 Commits (fa1bdfdda75a91640473125f9c7aaf33bc5f9b12)

Author SHA1 Message Date
Åke 228f63b50e
Merge pull request #2331 from MycroftAI/feature/voice-logs
Exception logging for AudioProducer and service reloader.
2019-09-30 11:15:32 +02:00
Åke Forslund d41fe97fd6 Add test case for the previously failing string 2019-09-30 11:02:29 +02:00
Åke Forslund 8f50217ee9 Merge branch 'bug/day-after-tomorrow' of https://github.com/ChanceNCounter/mycroft-core into ChanceNCounter-bug/day-after-tomorrow 2019-09-30 11:00:12 +02:00
Åke c8e51affec
Merge pull request #2334 from ChanceNCounter/gitignore-update
gitignore vscode workspace and html coverage data
2019-09-30 10:59:16 +02:00
Åke Forslund 9254ee0229 Remove duplicate load of skill settings
Instead of loading skill settings from disk twice, a copy is made into the
_initial_settings member after load.
2019-09-29 18:39:07 +02:00
ChanceNCounter a44167a3a4 Fixes index error in parse_en.extract_datetime()
==== Fixed Issues ====
#2333

====  Tech Notes ====
extract_datetime("day after tomorrow") attempted to check the previous
word. This resulted in an index error. Fixed by introducing a nested
condition. Seems to have solved it.

A similar bug is present in other parts of this file, such as line 843.

I have also discovered a number of other oddities, but they may be
outside the scope of this issue. At least Mycroft can now extract from
"day after tomorrow."

==== Localization Notes ====
I haven't checked parsers for any other languages.
2019-09-28 14:31:59 -07:00
ChanceNCounter 70b76f8396 gitignore vscode workspace and html coverage data 2019-09-28 14:00:27 -07:00
Chris Veilleux 364c3991e7 fixed a spelling error 2019-09-28 12:55:31 -05:00
Chris Veilleux eca875558a added log message and docstring to help demystify regex intent handling 2019-09-28 12:21:28 -05:00
Chris Veilleux f16989c2ec Add log messages that provide insight into the disambiguation behavior 2019-09-28 12:20:22 -05:00
Chris Veilleux 5980779e48 Refactored module level docstring and import ordering 2019-09-28 12:17:12 -05:00
David Wagner 86b6ef16e5 Log IOErrors 2019-09-27 22:25:54 +00:00
David Wagner 4536de7a06 Add Exception logging 2019-09-27 21:50:24 +00:00
Kris Gesling 87ee92d4f5
Merge pull request #2325 from forslund/bugfix/skill-tester-settings
Fix TestSettings
2019-09-27 12:13:36 +09:30
Chris Veilleux dfd776cab9 add a geolocation api for things like time and weather in requested locations 2019-09-26 16:24:11 -05:00
Åke 9cff2b8b0d
Merge pull request #2252 from cclauss/patch-2
Replace $IS_TRAVIS with builtin $CI
2019-09-26 16:42:51 +02:00
Åke Forslund a77216714d Replace new instance of IS_TRAVIS with CI 2019-09-26 16:32:38 +02:00
Christian Clauss 89a48b9f79 Force a rebuild 2019-09-26 16:32:38 +02:00
Christian Clauss 8fd4a32fb1 Remove definition of $IS_TRAVIS 2019-09-26 16:32:38 +02:00
Christian Clauss 121577487f Replace $IS_TRAVIS with builtin $CI
Updates dev_setup.sh and scripts/prepare_msm.sh
Fixes #2243
2019-09-26 16:31:46 +02:00
Åke bf38812ddf
Merge pull request #2297 from ChanceNCounter/prevent-repeating-dialog-lines
Cycle through dialog files to avoid repeating the same line
2019-09-26 16:23:08 +02:00
ChanceNCounter 30bd84bc85 Astd minor fix in core.dialog
Assign hardcoded value from MustacheDialogRenderer.render() to variable.
Fix its value to properly accommodate short dialog files.

Remove superfluous check on lines 104-105 of mycroft.dialog.init

====  Tech Notes ====
The new dialog logic, which prevents Mycroft from repeating itself, uses
a list of recently-spoken lines from the skill's .dialog files.
The size of this list is capped, currently at 3. However, .dialog files
can be shorter than 3 entries. An offset is used to accommodate this.

In the previous commit, the offset was wrong. Now it's 2.

====  Documentation Notes ====
If presented with a .dialog file containing 4 or more entries,
Mycroft will avoid saying the same one for at least a few rounds. If the
dialog file contains only 3 entries, Mycroft will avoid saying the same
line twice in a row. If only 1-2 entries are present in a .dialog file,
Mycroft will just pick one and say it, without retaining a history.
2019-09-26 16:20:43 +02:00
ChanceNCounter 4c8ce58f8e When able, cycle lines in .dialog files
Attempt to use a different line each time a .dialog file is invoked.

====  Tech Notes ====
MustacheDialogRenderer:
  - Keep a running list of recently spoken lines (up to 3)
  - If there's more than one line in a dialog file, don't repeat lines
  - Pop lines from the list when full or when dialog file is running low

====  Documentation Notes ====
Consider encouraging skill authors to include multi-line .dialog files,
as this will help to give Mycroft a more natural feel
2019-09-26 16:20:43 +02:00
Åke 7c7d7f47bb
Merge pull request #2262 from KishCom/bugfix/marytts-config-issue#2127
Fix missing param when setting up MaryTTS and how it validates a conn…
2019-09-26 16:14:53 +02:00
Andrew Kish 1d2cabd421 Fix missing param when setting up MaryTTS and how it validates a connection 2019-09-26 16:11:02 +02:00
Åke d30c50e752
Merge pull request #2283 from forslund/refactor/config-properties
Replace config properties to simpler instances
2019-09-26 14:51:56 +02:00
David Wagner 6bf66bf1f3
Merge pull request #2326 from MycroftAI/feature/remove-ntp-reboot
Remove ntp sync reboot
2019-09-25 14:53:13 -05:00
David Wagner 7cb1ca7282 Remove unused methods 2019-09-25 10:58:51 -05:00
Åke ee2c566218
Merge pull request #2317 from MycroftAI/feature/mic-watchdog
Add muted state to mic_level for mic watchdog
2019-09-25 13:02:00 +02:00
Åke Forslund 48e93f3504 Fix TestSettings
Updates the settings to work with the new ones
2019-09-25 13:01:45 +02:00
David Wagner b0c7d3072c Update text client parsing of mic_level file 2019-09-25 12:23:11 +02:00
David Wagner a66ec9388d Write mic level with mute 2019-09-25 11:54:22 +02:00
Åke f1a95e79cb
Merge pull request #2323 from forslund/bugfix/logline
Catch inspect error in logger
2019-09-25 07:53:02 +02:00
David Wagner 3fc389dbf3 Remove ntp sync reboot 2019-09-24 20:35:11 +00:00
Åke Forslund 39d0053493 Catch inspect error in logger
Inspect would in certain cases fail to find the line number. This
catches the exception and provides a basic name for the logger.
2019-09-24 20:58:34 +02:00
devs-mycroft b746f365c9 Version bump from 19.8.-1 to 19.8.0 2019-09-24 05:42:59 +00:00
Åke b6c1f5dd63
Merge pull request #2321 from forslund/feature/version-bump
Update version number in preparation for 19.08
2019-09-23 10:34:33 +02:00
Åke Forslund c8b33704ce Update version number in preparation for 19.08 2019-09-23 09:58:09 +02:00
Åke 6c228d00a9
Merge pull request #2320 from MycroftAI/bugfixes/skills-manifest
Bugfixes/skills manifest
2019-09-20 22:57:57 +02:00
Åke Forslund 62bba61da0 Make sure skill manifest is synced with backend
Send skill manifest all startups
2019-09-20 16:40:30 +02:00
Åke Forslund 347e8c9d1e Work on copy when managing the skills manifest
Fixes possible modification of input data when adding device uuid to the
manifest before sending.
2019-09-20 15:02:51 +02:00
Åke Forslund c6b1e3dc92 Make logging a little easier to decipher 2019-09-20 12:20:46 +02:00
Åke Forslund fcc4df6351 Clear msm device_skill_state before update/upload
Make sure to re-read the device_skill_state from disk before doing
changes so no information is lost.
2019-09-20 08:07:03 +02:00
Åke eddc3c45bc
Merge pull request #2318 from krisgesling/feature/yes-affirmative
add 'affirmative' to yes vocab
2019-09-19 20:24:07 +02:00
Åke Forslund 2cb2c56572 Install priority skills and save info
Use the msm.install method to install the priority skills so the skill
details are registered in the manifest
2019-09-19 20:07:21 +02:00
Åke Forslund e0cb16eab8 Clean up skill_manager exceptions
- BaseException -> Exception
- Remove unused vaiables
2019-09-19 20:07:21 +02:00
Åke Forslund c016903b77 Update skills manifest if a skill is reloaded.
When a skill is reloaded it is quite possible that a new skill gid has
been assigned to it and the backend needs to be informed of the change.
2019-09-19 20:07:21 +02:00
Åke 95a56373c3
Merge pull request #2319 from forslund/bugfix/orphaned-timers
Stop settingmeta upload on skill shutdown
2019-09-19 18:16:33 +02:00
Åke Forslund 3474d725b8 Stop settingmeta upload on skill shutdown
This makes sure to shutdown any Timer handling settingsmetadata upload
on skill shutdown to make sure no duplicates are running.

(Easily occurs when a skill gid doesn't match the one sent in skill
manifest)
2019-09-19 15:25:55 +02:00
Kris Gesling 6421509f35 add 'affirmative' to yes vocab 2019-09-19 10:47:20 +09:30