kz930 commented on PR #7645: URL: https://github.com/apache/texera/pull/7645#issuecomment-5431641154
Pushed one commit since the approvals (`8138e04`). The narrowing only reached the operators under `SklearnModelOpDesc`, so Linear Regression, Prediction and Testing were still outside it, which meant a model fitted with columns left out could be read with them back in. The rule now lives in one trait mixed in wherever a frame is handed to an estimator, and Prediction asks the model for `feature_names_in_` rather than re-deriving it. Re-requesting review, sorry for the extra round. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
