You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, the controller methods read the configuration file and pass this configuration dictionary to the business functions.
This creates a sprawl of difficult to maintain dictionary key references in sub-methods that can be avoided by just passing in these configuration options as individual arguments.
I originally did not want to deal with a large number of arguments from configuration, but I think unpacking the config dict is ultimately the better call after dealing with the codebase for a while.
This will entail a decent-sized refactor where I would want some overall-pipeline level tests available first.
The text was updated successfully, but these errors were encountered:
Currently, the controller methods read the configuration file and pass this configuration dictionary to the business functions.
This creates a sprawl of difficult to maintain dictionary key references in sub-methods that can be avoided by just passing in these configuration options as individual arguments.
I originally did not want to deal with a large number of arguments from configuration, but I think unpacking the config dict is ultimately the better call after dealing with the codebase for a while.
This will entail a decent-sized refactor where I would want some overall-pipeline level tests available first.
The text was updated successfully, but these errors were encountered: