Update skills-essential.txt

[JarbasAl]

update default skills list

should we add a list with OCP skills only?

what about GUI only skills? like homescreen

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (cf8b6b5) 62.72% compared to head (2433741) 63.11%.
Report is 1 commits behind head on dev.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #401      +/-   ##
==========================================
+ Coverage   62.72%   63.11%   +0.39%     
==========================================
  Files          14       14              
  Lines        2039     2039              
==========================================
+ Hits         1279     1287       +8     
+ Misses        760      752       -8     

Flag	Coverage Δ
unittests	63.11% <100.00%> (+0.39%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[emphasize]

rationale for ocp skills?
I see GUI skills for headless distributions.

[builderjer]

maybe a meta package for OCP and GUI? Instead of making it an option with core.

ovos-media-packages
ovos-gui-packages

Then users can just install that meta package and have what they need. It keeps the minimal amount in core requirements itself

[mikejgray]

LGTM, is the OVOS alerts skill ready for prime time? I thought it still had some work to do

[mikejgray]

Actually had a look at the tests, this would officially break compat with Python 3.7 it looks like, and possibly others (the tests were cancelled before we could see).

[emphasize]

LGTM, is the OVOS alerts skill ready for prime time? I thought it still had some work to do

Needs translation other than german and that is hard to come by and needs someone who is a heavy user, not only the usual example test.
Everything other than "en_us" (and "de-de" ofc) is autotranslated and bound to failure. But that's the way it is.
Its Adapt intent, so you have more freedom how to form a sentence, but the parser may butcher the event/reminder name.

A reason why list todos are recorded one by one (looping get_response) without parsing as this would be a mess.

[builderjer]

I don't mind this, it does cut down the "essential" skills for sure. I still think it may be better to have "meta" packages.

I installed the basic core with the "essential" skills. Later, I add a screen and want to add the GUI. I could find the repo ovos-gui-package clone and install it, and it would provide all of the skills and plugins needed

[builderjer]

Why is this a desktop skill? Could it not be used on a headless device?

[mikejgray]

This is a good point - the skill runtime_requirements do specify it doesn't require a GUI

[builderjer]

I still don't think this should be a desktop skill unless I am missing something.

But being commented out, does not apply yet

[JarbasAl]

its commented out because its not published, needs automations in the repo

ill merge for now until that is done as it should not block 0.0.8

[JarbasAl]

https://github.com/OpenVoiceOS/skill-ovos-randomness should also be added

[mikejgray]

https://github.com/OpenVoiceOS/skill-ovos-randomness should also be added

Need to get it on PyPi first

[builderjer]

Not sure it should be included in the defaults. It is a great extra, but I think the default should be kept to a minimum

[builderjer]

A comment on ovos-skill-dictation, and still not sure on adding ovos-skill-randomness either to the defaults. I really like the idea of keeping things small and getting a skill store going.

[builderjer]

I still don't think this should be a desktop skill unless I am missing something.

But being commented out, does not apply yet

[builderjer]

And looking through things, is this needed as a requirement when we don't require precise?

[builderjer]

I can approve this with the commented skills as is.