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
In src/forge/common.py there is a function normalize_config. It is used to normalize the configuration values, but it currently does too many things.
The Solution
Separate normalize_config into parse_additional_config and normalize_config where parse_additional_config will handle the extra functionality in processing the additional config. Replace the function calls to normalize_config where it calls the additional config part of the function.
Notes
The additional config part of normalize_config is called in main.py around line 140 and in yaml_loader.py around 230.
The text was updated successfully, but these errors were encountered:
The Issue
In
src/forge/common.py
there is a functionnormalize_config
. It is used to normalize the configuration values, but it currently does too many things.The Solution
Separate
normalize_config
intoparse_additional_config
andnormalize_config
whereparse_additional_config
will handle the extra functionality in processing the additional config. Replace the function calls tonormalize_config
where it calls the additional config part of the function.Notes
The additional config part of
normalize_config
is called inmain.py
around line 140 and inyaml_loader.py
around 230.The text was updated successfully, but these errors were encountered: