hequn8128 commented on issue #11220: [FLINK-16249][python][ml] Add interfaces for Params, ParamInfo and WithParams URL: https://github.com/apache/flink/pull/11220#issuecomment-591807495 > @hequn8128 Thanks for the PR. > > 1. I noticed that there are two modules "ml" and "mllib" introduced. I'm wondering if one module such as ml is enough? @dianfu This sounds like a good idea. We can add api and lib under the `pyflink.ml` package. The package under the pyflink should be an entire one, e.g., `pyflink.table`. @becketqin @walterddr What's your opinion on this. If we reach a consensus on this, we may need an extra vote since it is different from the FLIP wiki page.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
