aglinxinyuan opened a new issue, #4826: URL: https://github.com/apache/texera/issues/4826
### Task Summary `common/workflow-operator/src/main/scala/org/apache/texera/amber/operator/sklearn` defines 24 concrete `SklearnClassifierOpDesc` subclasses and 26 concrete `SklearnTrainingOpDesc` subclasses, each overriding `getImportStatements` and `getUserFriendlyModelName`. These two strings drive the generated Python code (and thus actual model selection) and the user-facing UI label, but neither is currently asserted anywhere — a typo would silently misroute either or both. Add `SklearnOpDescRegistrySpec`: - Pin the `(import statement, user-friendly name)` pair for every concrete `SklearnClassifierOpDesc` (24 entries) - Pin the `(import statement, user-friendly name)` pair for every concrete `SklearnTrainingOpDesc` (26 entries) - Verify the `SklearnClassifierOpDesc` base default returns empty strings before subclass override - Verify the `SklearnTrainingOpDesc` base default uses RandomForest until overridden - Smoke test `generatePythonCode` for a representative classifier and training subclass to verify the import string is embedded ### Priority P3 – Low ### Task Type - [x] Testing / QA -- 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]
