Refactor/inputstate/combine validation #769
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kmantel
added a commit
that referenced
this pull request
May 1, 2018
* devel: Feat/projections/masked mapping projection (#775) adding exceptions to get_current_function_param and get_current_mechanism_param - not intended for looking up 'variable' Feat/projections/masked mapping projection (#771) • LCA (#770) • LCA (#769) Refactor/mechanism/transfermechanism (#768) Fix default_variable bug (#767) Feat/inputstate/combine (#766) Refactor/context/source (#765) Defaults: ensure that function attr appears when examining Defaults Rewrite Function instantiation: Component: create fewer instances during init when function arg is a class LinearMatrix: make keyword method static to reflect how it is used refactor/rename _variable_not_specified to be more generic, using enum Defaults: add enum to denote flexibility of assignments Component: add class method to get new param class defaults Defaults: add ClassDefaults.function to several classes, and use it as fallback utilities: add function to detect if obj is an instance or a subclass of a class Component: override __deepcopy__ method to use shared items parsing: add method to parse function variable from variable imports: pycharm-optimize to avoid unnecessary circularities Params: copy dict/ROODs nested in params arguments to avoid side effects Refactor/context/set and test (#762) fixing bug in looking up target values for learning which caused first target value to be repeated LinearCombination, Reduce: handle negative exponents during init (avoid zero division) Functions: simplify noise validation adding a note to the ddm plot() documentation to clarify that the plot does not correspond to the actual results of the mechanism fixing drift diffusion integrator bug - returned threshold even when it crossed the negative threshold Fix/misc/misc (#759) PredictionErrorMechanism: correct infinite recursion in _execute Docs/context/context (#757) • Mechanism (#756) Refactor/mechanisms/ execute (#754) LearningAuxilliary: rename to LearningAuxiliary (correct double l typo) scheduling: decouple Scheduler and Condition, allow multiple execution_ids utilites: get method to prune unused args from a function from composition branch Feat/mechanism/input target lable dicts (#752) testing: Resolve leftover merge conflicts, fixes #747 testing: correct pytest setup ovewrite, losing some settings Feat/mechanism/input target label dicts (#751) Scheduling: fix bug where termination conditions persisted across calls to run tests,function/LinearCombination: Add tests with absent parameters test,function/LinearCombination: Rename second test function to prevent overwriting results • Context (#746) Refactor/context/deprecate init status (#745) Refactor/context/structured (#744) Fix/function/stability (#743) • IntegratorMechanism (#742)
kmantel
added a commit
that referenced
this pull request
May 1, 2018
* devel: Feat/projections/masked mapping projection (#775) adding exceptions to get_current_function_param and get_current_mechanism_param - not intended for looking up 'variable' Feat/projections/masked mapping projection (#771) • LCA (#770) • LCA (#769) Refactor/mechanism/transfermechanism (#768) Fix default_variable bug (#767) Feat/inputstate/combine (#766) Refactor/context/source (#765) Defaults: ensure that function attr appears when examining Defaults Rewrite Function instantiation: Component: create fewer instances during init when function arg is a class LinearMatrix: make keyword method static to reflect how it is used refactor/rename _variable_not_specified to be more generic, using enum Defaults: add enum to denote flexibility of assignments Component: add class method to get new param class defaults Defaults: add ClassDefaults.function to several classes, and use it as fallback utilities: add function to detect if obj is an instance or a subclass of a class Component: override __deepcopy__ method to use shared items parsing: add method to parse function variable from variable imports: pycharm-optimize to avoid unnecessary circularities Params: copy dict/ROODs nested in params arguments to avoid side effects Refactor/context/set and test (#762) fixing bug in looking up target values for learning which caused first target value to be repeated LinearCombination, Reduce: handle negative exponents during init (avoid zero division) Functions: simplify noise validation adding a note to the ddm plot() documentation to clarify that the plot does not correspond to the actual results of the mechanism fixing drift diffusion integrator bug - returned threshold even when it crossed the negative threshold Fix/misc/misc (#759) PredictionErrorMechanism: correct infinite recursion in _execute Docs/context/context (#757) • Mechanism (#756) Refactor/mechanisms/ execute (#754) LearningAuxilliary: rename to LearningAuxiliary (correct double l typo) scheduling: decouple Scheduler and Condition, allow multiple execution_ids utilites: get method to prune unused args from a function from composition branch Feat/mechanism/input target lable dicts (#752) testing: Resolve leftover merge conflicts, fixes #747 testing: correct pytest setup ovewrite, losing some settings Feat/mechanism/input target label dicts (#751) Scheduling: fix bug where termination conditions persisted across calls to run tests,function/LinearCombination: Add tests with absent parameters test,function/LinearCombination: Rename second test function to prevent overwriting results • Context (#746) Refactor/context/deprecate init status (#745) Refactor/context/structured (#744) Fix/function/stability (#743) • IntegratorMechanism (#742)
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
• InputState
moved validation and implementation of combine option from init
to _validate_params and _instantiate_function