otoole
otoole copied to clipboard
[FEATURE]: otoole can not reach `ReadStrategy._get_missing_input_dataframes(...)`
Feature Request
The ._get_missing_input_dataframes(...) method is meant to align the input_data with the config file data. It will add empty dataframes if the user has not supplied data. However, the method ReadStrrategy._compare_read_to_expected(...) is often called before ._get_missing_input_dataframes(...) in the concrete classes. ._compare_read_to_expected(...) will raise a OtooleNameMismatchError if the input data and config data do not match (which is not entirely clear from the name OtooleNameMismatchError).
Is your feature related to a bug?
#169
Suggested Solution
I guess we could:
- Get rid of the
._compare_read_to_expected(...)method, and just raise a warning if the input data and config data dont match - Change the error handling to a flag option (see issue #169)
- Get rid of the
._get_missing_input_dataframes(...)method and strictly enforce data consistency (which may be tedious for legacy models)
Additional Info
No response