-
Notifications
You must be signed in to change notification settings - Fork 32
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/lccontrolmechanism_objectivemechanism #3134
Conversation
…Link into devel # Conflicts: # Scripts/Models (Under Development)/EGO/Using EMComposition/ScriptControl.py # Scripts/Models (Under Development)/EGO/Using EMComposition/TestParams.py
…Link into devel # Conflicts: # Scripts/Models (Under Development)/EGO/Using EMComposition/CSW/ScriptControl.py
…Link into fix/add_obj_mech_warning
- suppress warning about adding CONTROL_OBJECTIVE NodeRole if ObjectiveMechanism is already associated with a ControlMechanism
…Link into fix/add_obj_mech_warning
• lccontrolmechanism.py - _instantiate_control_signals: complete implementation of ALL
• lccontrolmechanism.py - _instantiate_control_signals: complete implementation of ALL
• lccontrolmechanism.py - _instantiate_control_signals: complete implementation of ALL
…ithub.com/PrincetonUniversity/PsyNeuLink into fix/lccontrolmechanism_objectivemechanism
test_lc_control_monitored_and_modulated_mechanisms_composition(): augmented to test implementation of ObjectiveMechanism
@@ -2912,7 +2912,7 @@ | |||
from psyneulink.core.components.mechanisms.modulatory.modulatorymechanism import ModulatoryMechanism_Base | |||
from psyneulink.core.components.mechanisms.processing.compositioninterfacemechanism import CompositionInterfaceMechanism | |||
from psyneulink.core.components.mechanisms.processing.objectivemechanism import ObjectiveMechanism | |||
from psyneulink.core.components.mechanisms.processing.processingmechanism import ProcessingMechanism | |||
from psyneulink.core.components.mechanisms.processing.processingmechanism import ProcessingMechanism, ProcessingMechanism_Base |
Check notice
Code scanning / CodeQL
Cyclic import Note
psyneulink.core.components.mechanisms.processing.processingmechanism
This PR causes the following changes to the html docs (ubuntu-latest-3.11):
See CI logs for the full diff. |
assert T_1.parameter_ports[pnl.SLOPE].mod_afferents[0] in LC.control_signals[0].efferents | ||
assert T_2.parameter_ports[pnl.SLOPE].mod_afferents[0] in LC.control_signals[0].efferents | ||
|
||
result = C.run(inputs={T_1:[1,2]})#, T_2:[3,4,5] |
Check notice
Code scanning / CodeQL
Unused local variable Note test
This PR causes the following changes to the html docs (ubuntu-latest-3.11):
See CI logs for the full diff. |
This PR causes the following changes to the html docs (ubuntu-latest-3.11):
See CI logs for the full diff. |
• lccontrolmechanism.py
_instantiate_objective_mechanism(): implement override of ControlMechanism to assign CombineMeans as function of default ObjectiveMechanism
• test_control_mechanism.py