MycroftAI / mycroft-core

Mycroft Core, the Mycroft Artificial Intelligence platform.
https://mycroft.ai
Apache License 2.0
6.51k stars 1.27k forks source link

Speech recognition in Brazilian Portuguese #2977

Closed tiagochiavericosta closed 3 years ago

tiagochiavericosta commented 3 years ago

Description

Speech recognition in Brazilian Portuguese

How to test

Brazilian Portuguese speech recognition tested and working correctly. Recognition model using software produced by Brazilian Universities www.eva.fmrp.usp.br (USP) and www.portal.ufpa.br (UFPA)

Contributor license agreement signed?

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

devops-mycroft commented 3 years ago

Hello, @tiagochiavericosta, thank you for helping with the Mycroft project! We welcome everyone into the community and greatly appreciate your help as we work to build an AI for Everyone.

To protect yourself, the project, and users of Mycroft technologies we require a Contributor Licensing Agreement (CLA) before accepting any code contribution. This agreement makes it crystal clear that along with your code you are offering a license to use it within the confines of this project. You retain ownership of the code, this is just a license.

Please visit https://mycroft.ai/cla to initiate this one-time signing. Thank you!

devops-mycroft commented 3 years ago

Voight Kampff Integration Test Failed (Results). Mycroft logs are also available: skills.log, audio.log, voice.log, bus.log, enclosure.log

tiagochiavericosta commented 3 years ago

Why changing Mycroft to EVA (same question for the Git repository)?

I submitted the voice template in Brazilian Portuguese to the Mycroft staff. But I also made changes to my fork, which I will translate entirely to Brazilian Portuguese (called EVA), I must have gone wrong in something or created a new repository instead of working on top of the fork. Please ignore these changes and just accept the voice recognition models for Brazilian Portuguese (which I used pull request)

ChanceNCounter commented 3 years ago

FWIW, I think most of the community members who work on language support would just as soon work toward full support in a given language, rather than seeing language-specific forks. The biggest obstacles are, indeed, quality TTS and STT, and then skills need translating.

That's stuff a fork has to do anyway, but if we keep improving Mycroft itself in the same way, every user can benefit and the localization efforts can snowball.

(edit: ten internets to anyone who can tell me why i keep getting an unread notification for this, my own comment)

krisgesling commented 3 years ago

Hey Tiago, apologies the bot got confused, we have got your CLA and you've been added to our list of awesome humans.

We're trying to pull these packages out to use the new plugin system rather than adding more and more to mycroft-core. So I'm going to suggest we hold this PR until that plugin is ready.

Can I also check whether this model carries any existing licensing with it? And confirm that it's your work so you have the authority to determine the license under which it's released?

By including it in this repository with no other information, you are releasing it under an Apache 2.0 license. If that's not your intention, you could package it up as it's own plugin instead.

tiagochiavericosta commented 3 years ago

In the project I used parts of the GPL license model. You can put it under the Apache license (which keep the source open). Just mention the EVA Project (eva.frmp.usp.br), coordinated by Professor Gleice da Silva Castro Perdoná (https://gleici.fmrp.usp.br). I will make more contributions in the translation into Portuguese. I am about to release the Linux operating system integrated with the integrated virtual assistant. Anything I communicate to you.

Abraço, Tiago Chiaveri da Costa


De: Kris Gesling @.> Enviado: quarta-feira, 25 de agosto de 2021 01:57 Para: MycroftAI/mycroft-core @.> Cc: Tiago Chiaveri @.>; Mention @.> Assunto: Re: [MycroftAI/mycroft-core] Speech recognition in Brazilian Portuguese (#2977)

Hey Tiago, apologies the bot got confused, we have got your CLA and you've been added to our list of awesome humanshttps://github.com/mycroftAI/contributors.

We're trying to pull these packages out to use the new plugin systemhttps://mycroft-ai.gitbook.io/docs/mycroft-technologies/mycroft-core/plugins rather than adding more and more to mycroft-core. So I'm going to suggest we hold this PR until that plugin is ready.

Can I also check whether this model carries any existing licensing with it? And confirm that it's your work so you have the authority to determine the license under which it's released?

By including it in this repository with no other information, you are releasing it under an Apache 2.0 license. If that's not your intention, you could package it up as it's own plugin instead.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/MycroftAI/mycroft-core/pull/2977#issuecomment-905183009, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAXZAPVTCLOXZZSAS57OY7DT6RZ4RANCNFSM5CBTGYGA. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

krisgesling commented 3 years ago

Hi Tiago, we can't accept any GPL licensed material into mycroft-core. That would arguably require us to re-license the entire codebase to be GPL as well.

I'd suggest packaging this up as a plugin. That will allow you to define the licensing, terms, and attribution that is required by your institution and can still be used on any Mycroft installation.

For more information on our plugin system, see our docs here: https://mycroft-ai.gitbook.io/docs/mycroft-technologies/mycroft-core/plugins

If you have any questions about it, a good place to get help is the Dev channel on Mycroft Chat.