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

Move from scripts to classes #26

Open
pjf opened this issue Feb 10, 2014 · 2 comments
Open

Move from scripts to classes #26

pjf opened this issue Feb 10, 2014 · 2 comments

Comments

@pjf
Copy link
Owner

pjf commented Feb 10, 2014

Right now all end-points and actions are in scripts. There are a lot of them. They're cluttered. They don't inherit from any base class, or consume any common role.

We should fix this, because it will reduce a lot of boilerplate, and will make it much easier to write extensions. At the very least, I expect we'll want roles (or maybe classes) for:

  • Exobrain::Source (produces Exobrain::Measurement:: packets)
  • Exobrain::Action (consumes measurements, produces intents)
  • Exobrain::Sink (consumes Exobrain::Intent:: packets)

Our scripts can then be moved to classes which consume these roles. AFAIK Ubic can start/stop classes just fine. :)

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/1406284-move-from-scripts-to-classes?utm_campaign=plugin&utm_content=tracker%2F347315&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F347315&utm_medium=issues&utm_source=github).
@pjf
Copy link
Owner Author

pjf commented Feb 25, 2014

This is partially done, but in Exobrain::Agent and friends.

Leaving this ticket open because we want to move all the old scripts across to class-based agents.

@pjf
Copy link
Owner Author

pjf commented Mar 26, 2014

This is even more done. Leaving open pending me writing actual docs.

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

No branches or pull requests

1 participant