-
Notifications
You must be signed in to change notification settings - Fork 31
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
Feat/ocm/state feat dict #2291
Feat/ocm/state feat dict #2291
Conversation
…rincetonUniversity/PsyNeuLink into refactor/ocm/state_input_ports
- _update_state_input_ports_for_controller: handle nested input nodes
_update_state_input_ports_for_controller: fixed bug with > 1 INPUT node in Composition
…rincetonUniversity/PsyNeuLink into refactor/ocm/state_input_ports
…rincetonUniversity/PsyNeuLink into refactor/ocm/state_input_ports
- _instantiate_monitor_for_control_input_ports -> _parse_monitor_control_input_ports - refactored to support allow_probes option on ocm
- _instantiate_monitor_for_control_input_ports -> _parse_monitor_control_input_ports - refactored to support allow_probes option on ocm
- _instantiate_monitor_for_control_input_ports -> _parse_monitor_control_input_ports - refactored to support allow_probes option on ocm
__init__: move controller to after add_nodes and add_linear_pathway
…Link into feat/ocm/state_feat_dict � Conflicts: � psyneulink/core/compositions/composition.py
- _parse_state_feature_specs(): set InputPort.default_input=DEFAULT_VARIABLE if state_feature specification is numerical
- _parse_state_feature_specs(): set InputPort.default_input=DEFAULT_VARIABLE if state_feature specification is numerical
…onUniversity/PsyNeuLink into feat/ocm/state_feat_dict
_parse_state_feature_spec: fix bug with attempt to get nodes of CFA
- _update_state_input_ports_for_controller: error message edits • test_control.py: - test_ocm_state_feature_specs_and_warnings_and_errors: add test for numerical assignment of state_feature
- test_ocm_state_and_state_dict: numerical assignment to state_feature
- test_default_input: passes (but has comments with some desired refactoring
…Link into feat/ocm/state_feat_dict
This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):
See CI logs for the full diff. |
1 similar comment
This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):
See CI logs for the full diff. |
This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):
See CI logs for the full diff. |
- add_node(): if NodeRole.INTERNAL specified, assign internal_only to all InputPorts - _determine_node_roles(): if if NodeRole.INTERNAL specified, preclude NodeRole.INPUT
…onUniversity/PsyNeuLink into refactor/comp/internal_node
- add_node(): if NodeRole.INTERNAL specified, assign internal_only to all InputPorts - _determine_node_roles(): if if NodeRole.INTERNAL specified, preclude NodeRole.INPUT
This pull request introduces 2 alerts when merging 4ccaa7a into a64bbc5 - view on LGTM.com new alerts:
|
- add_node(): if NodeRole.INTERNAL specified, assign internal_only to all InputPorts - _determine_node_roles(): if if NodeRole.INTERNAL specified, preclude NodeRole.INPUT
This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):
See CI logs for the full diff. |
This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):
See CI logs for the full diff. |
This pull request introduces 2 alerts when merging 7f950fb into a64bbc5 - view on LGTM.com new alerts:
|
• composition.py:
• inputport.py:
• port.py:
• optimizationcontrolmechanism.py:
set InputPort.default_input=DEFAULT_VARIABLE if state_feature specification is numerical
• test_control.py:
• test_input_ports.py