lindong28 opened a new pull request #10:
URL: https://github.com/apache/flink-ml/pull/10


   ## What is the purpose of the change
   
   This PR updates the parameter-related clsses according to 
[FLIP-174](https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=181311361).
   
   ## Brief change log
   
   This PR mades the following changes:
   - Updated the `WithParams` interface.
   - Removed the `ParamInfoFactory` and the `Params` classes.
   - Replaced the `ParamInfo` class with the `Param` class.
   - Added a few pre-defined `Param` subclasses  for common types such as int, 
double, float, string, double array etc. 
   - Added `ParamValidators` to provide factory methods for common validation 
functions on numerical values.
   
   ## Verifying this change
   
   The features added in this test are verified by the following unit tests:
   - The `StageTest` verifies parameter set/get operations and stage save/load 
operations.
   - The `PipelineTest` verifies Pipeline::fit and PipelineModel::transform. 
And it also verifies save/load of Pipeline and PipelineModel.
   
   ## Does this pull request potentially affect one of the following parts:
   
   - Dependencies (does it add or upgrade a dependency): (no)
   - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (yes)
   
   ## Documentation
   
   - Does this pull request introduce a new feature? (yes)
   - If yes, how is the feature documented? (Java doc)


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to