-
Notifications
You must be signed in to change notification settings - Fork 87
Consolidate ActorOutput and introduce automatic docstring #599
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
Conversation
…_for_actoroutput # Conflicts: # docs/source/user_guide/user_guide_reference_actors.rst
…ssing interface class)
|
UPDATE: I am still fixing some structural stuff with the actor output handling in this PR, but I am almost done. Hope this can be merged tomorrow. |
…mpatible with pickling)
…_with_random_seed.py (see PR #574)
|
Some background on this PR (will integrate this in the dev doc later). Technical level: GATE digests all this automatically and issues a GateImplementationError if anything is missing. |
This PR reworks the way actor output classes are associated with actor classes.
Previously, this was done at an instance level. Now, each actor class gets its own copies of the actor output classes it needs and default values of the actor output can be set at a class level per actor class.
The mechanism makes use of the existing class factory mechanism used for all GateObjects and extends it for all actors.
The actor class docstrings now automatically contain information about the user output (the interfaces to the output, to be precise) and well as the default settings, e.g. active, write_to_disk, etc. Sphinx/autodoc pick this up automatically.