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

Fix/system/learning ends before terminal #685

Merged
merged 4 commits into from
Mar 2, 2018

Conversation

jdcpni
Copy link
Collaborator

@jdcpni jdcpni commented Mar 2, 2018

• System
_instantiate_learning_graph(): fixed bug in which learning failed to
be instantiated for a System in which the last Mechanism in a
learning sequence was not the TERMINAL Mechanism of the System.

• Scripts/Examples
Rumelhart Semantic Network: updated with final Projection (that doesn't learn) and Mechanism (using step function)

jdcpni added 4 commits March 1, 2018 23:14
  _instantiate_learning_graph(): fixed bug in which learning failed to
    be instantiated for a System in which the last Mechanism in a
    learning sequence was not the TERMINAL Mechanism of the System.
  _instantiate_learning_graph(): fixed bug in which learning failed to
    be instantiated for a System in which the last Mechanism in a
    learning sequence was not the TERMINAL Mechanism of the System.
  Rumelhart Semantic Network: updated with step function
@jdcpni jdcpni merged commit a75aeb0 into master Mar 2, 2018
KristenManning added a commit that referenced this pull request Mar 2, 2018
KristenManning added a commit that referenced this pull request Mar 2, 2018
…tem/learning-ends-before-terminal

Revert "Fix/system/learning ends before terminal (#685)"
jdcpni added a commit that referenced this pull request Apr 13, 2018
* Fix/system/learning ends before terminal (#685)

* • System
  _instantiate_learning_graph(): fixed bug in which learning failed to
    be instantiated for a System in which the last Mechanism in a
    learning sequence was not the TERMINAL Mechanism of the System.

* • System
  _instantiate_learning_graph(): fixed bug in which learning failed to
    be instantiated for a System in which the last Mechanism in a
    learning sequence was not the TERMINAL Mechanism of the System.

* • Scripts/Examples
  Rumelhart Semantic Network: updated with step function

* Revert "Fix/system/learning ends before terminal (#685)"

This reverts commit a75aeb0.

* • Environment
  Run: removed _parse_target_labels (they are just the input_labels
       for a TARGET Mechanism

• Mechanism
  docstring revs for INPUT_LABELS_DICDT and OUTPUT_LABELS_DICT

* -

* -

* -

* -

* -

* • Component
  - _execute:  added **kwargs argument, passed to call to function
               to accomodate LearningMechanism that uses this

• AutoassociativeLearningMechanism
  - _execute calls super(LearningMechanism, self) to skip Learning

• LearningMechanism:
  _ execute calls super()._execute with **kwargs

* • Component
  - _execute:  added **kwargs argument, passed to call to function
               to accomodate LearningMechanism that uses this

• AutoassociativeLearningMechanism
  - _execute calls super(LearningMechanism, self) to skip Learning

• LearningMechanism:
  _ execute calls super()._execute with **kwargs

• PrecictionMechanism:
  _execute calls super()._execute

* • Component
  - _execute:  added **kwargs argument, passed to call to function
               to accomodate LearningMechanism and EVCControlMechanism,
               the functions for which expect additional arguments

• AutoassociativeLearningMechanism
  - _execute calls super(LearningMechanism, self) to skip Learning

• LearningMechanism:
  _ execute calls super()._execute with **kwargs

• PrecictionMechanism:
  _execute calls super()._execute

* • Component
  - _execute:  added **kwargs argument, passed to call to function
               to accomodate LearningMechanism and EVCControlMechanism,
               the functions for which expect additional arguments

• AutoassociativeLearningMechanism
  - _execute calls super(LearningMechanism, self) to skip Learning

• LearningMechanism:
  _ execute calls super()._execute with **kwargs

• PrecictionMechanism:
  _execute calls super()._execute
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant