-
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
refactor/emcomposition_softmax_after_dot #3095
Conversation
…Link into devel # Conflicts: # Scripts/Models (Under Development)/EGO/Using EMComposition/ScriptControl.py # Scripts/Models (Under Development)/EGO/Using EMComposition/TestParams.py
- move softmax operation to after combining weighted dot products
…thub.com/PrincetonUniversity/PsyNeuLink into refactor/emcomposition_softmax_after_dot
…thub.com/PrincetonUniversity/PsyNeuLink into refactor/emcomposition_softmax_after_dot
# if DISPLAY_MODEL is not None: | ||
# model.show_graph(**DISPLAY_MODEL) |
Check notice
Code scanning / CodeQL
Commented-out code Note
# elif self.concatenate_queries_node: | ||
# input_source = self.concatenate_queries_node | ||
# proj_name =f'{CONCATENATE_QUERIES_NAME} to {SOFTMAX_NODE_NAME}' |
Check notice
Code scanning / CodeQL
Commented-out code Note
This PR causes the following changes to the html docs (ubuntu-latest-3.11):
See CI logs for the full diff. |
…Link into refactor/emcomposition_softmax_after_dot
…thub.com/PrincetonUniversity/PsyNeuLink into refactor/emcomposition_softmax_after_dot
This PR causes the following changes to the html docs (ubuntu-latest-3.11):
See CI logs for the full diff. |
• emcomposition.py
- move softmax operation to after combining weighted dot products
- update use_gating_for_weighting and softmax CONTROL option to work with refactoring
- validate that use_gating_for_weighting is False when learn_field_weights is True
- docstring mods