Closed ChristyKoh closed 1 year ago
Since we require all the arguments from Extract within Eval, and the model name is a required positional argument for the Extract dataclass, it looks like this change would require a lot of refactoring for a little convenience.
extract model name from config instead of making user type it into cmdline