Skip to content
This repository was archived by the owner on Sep 8, 2024. It is now read-only.

Conversation

andlo
Copy link
Contributor

@andlo andlo commented Jan 6, 2020

Description

  • Adding a setting in mycroft.conf for padatious single:thread
  • adding use of setting in padatious

To make it posible to use remote debugging there is need for at setting to turn off and on singlethread in padatious. This I think were done best by a mycroft.conf setting.

When (if) this is merged I have a remote-debug skill that uses PTVSD to enable remote debug :)

How to test

Change setting for padatious single_thread in mycroft.conf and see that it changes how padatios run.

Se in log that single_thread is true or fals acording to setting in mycroft.conf

 19:40:43.542 | INFO     | 20607 | mycroft.skills.padatious_service:train:89 | Training... (single_thread=True)               

Contributor license agreement signed?

CLA [ x ] (Whether you have signed a CLA - Contributor Licensing Agreement

@pep8speaks
Copy link

pep8speaks commented Jan 6, 2020

Hello @andlo! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2020-01-07 16:53:51 UTC

@devs-mycroft devs-mycroft added the CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) label Jan 6, 2020
@forslund
Copy link
Collaborator

forslund commented Jan 7, 2020

Thanks @andlo, looks nice and simple! If you fix the pep8 issues we can merge this afterwards.

@forslund
Copy link
Collaborator

forslund commented Jan 7, 2020

Thanks @andlo. Looks great. Merging.

@forslund forslund merged commit 4ab39bf into MycroftAI:dev Jan 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants