Skip to content
This repository has been archived by the owner on Jul 21, 2024. It is now read-only.

Reduce brain size and AIML clutter #9

Open
jrwarwick opened this issue May 21, 2018 · 2 comments
Open

Reduce brain size and AIML clutter #9

jrwarwick opened this issue May 21, 2018 · 2 comments

Comments

@jrwarwick
Copy link

While reviewing AIML files I found several that seemed to be not such a great fit for Mycroft, and others that were of low quality. Moreover, I noticed in some of the current PRs that brain size is a concern. 3 birds with one stone: delete some AIML files; following are my recommendations.
Games ( cool hack, but probably should be separate skills, if it would even work with Mycroft's AIML)

  • fallback-aiml/aiml/battledome.aiml
  • fallback-aiml/aiml/stories.aiml
  • fallback-aiml/aiml/hangman.aiml
  • fallback-aiml/aiml/highroller.aiml
  • fallback-aiml/aiml/luckyslots.aiml
  • fallback-aiml/aiml/poker.aiml
  • fallback-aiml/aiml/zbert.aiml
  • fallback-aiml/aiml/numberdrop.aiml
  • fallback-aiml/aiml/tictactoe.aiml

Mitsuku specific:

  • fallback-aiml/aiml/warnings.aiml

and last, and quite large:
fallback-aiml/aiml/mp#.aiml
(where # is an integer number) series is mostly trash. I'd recommend removing and if some folks in the community that are not particularly comfortable with programming want to make some kind of contribution, they could go back and cherry pick (and correct!) the contents of these files.

For some helpful descriptions of the source AIML files: https://github.com/pandorabots/Free-AIML

@EazyAlvaro
Copy link
Contributor

#13 is my starting proposal

@jrwarwick
Copy link
Author

Thank you, @EazyAlvaro that was a great start (a year ago!). I think it needs even more clean up. Maybe someone will pick this up for upcoming Hacktoberfest or something. Perhaps this is a candidate for an Hacktoberfest Label.

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

No branches or pull requests

2 participants