Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed and added precise choose script #25

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

mikhaildmitrienko
Copy link
Contributor

Fixed bug that prevented tag files written by precise-choose from being read and added precise choose as executable.

Merge changes from mycroft-precise
Changed the maximum possible amount of units for the optimization to return to 70 units.
removed a redundant line of code. print("False positive: ", false_positives * 100, "%") is already printed so print(false_positives) isn't needed.
Changed spacing between file id and wakeword status from four spaces to a tab so that precise can read the newly created tags
print('False negative ', (i + 1), ' of ', (len(false_negatives_array)) + 1)
user_input = input('Enter y if wakeword, enter n for not wakeword \n')
time.sleep(5)
false_negatives_array[i] = false_negatives_array[i].lstrip('/Users/madmitrienko/wakewords/files/')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to generalize this and pass it to the script as a variable or an environmental setting.

chrisveilleux added a commit that referenced this pull request Apr 14, 2020
@ShamoX
Copy link

ShamoX commented Jul 21, 2022

Hello,

Since I need that project, I decided to fork and advance. Sorry for the inconvenience but it can be reversed if Mycroft wants of course).

You will find your PR already integrated to the fork here.

Again, sorry for the inconvenient. I created a thread on Mycroft community forum to discuss this.

@joshua-montgomery is this still relevant ? It needs some works anyway for it to be integrated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants