[issue44151] Improve parameter names and return value ordering for linear_regression
Zachary Kneupper added the comment: > Any objections to linear_regression(x, y) -> (slope, intercept)? I think `linear_regression(x, y)` would be intuitive for a wide range of users. Just to clarify, is the proposal to return a regular tuple instead of named tuple? Would we do this: return (slope, intercept) and not do this: return LinearRegression(intercept=intercept, slope=slope) and not do this: return Line(intercept=intercept, slope=slope) ? -- ___ Python tracker <https://bugs.python.org/issue44151> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue37439] Add random.binomialvariate()
Zachary Kneupper added the comment: Is adding random.binomialvariate() still under consideration? I would be willing to work on this addition to Lib/random.py, as well as the accompanying tests and docs. Should I use the type of algorithm hinted at by Mark on 2019-07-01 17:56? -- nosy: +zkneupper ___ Python tracker <https://bugs.python.org/issue37439> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue44151] Improve parameter names and return value ordering for linear_regression
Zachary Kneupper added the comment: > The ML world has collapsed on the terms X and y. (With that > capitalization). The ML community will probably use 3rd party packages for their linear regressions in any case. In my estimation, the ML community would be comfortable with any of these pairs of terms: Fine: + regressor, dependent_variable + independent_variable, dependent_variable + x, y Bad: + X, y <- this wouldn't makes sense here since the first argument is always a vector and is never a matrix. Often, capital letters indicate matrices, and lower case letters indicate vectors (or scalars). The reason that X is often capitalized is because it indicates that X is an m-by-n matrix of several independent variables; whereas y is lowercase because it is a single vector for the dependent variable. Since this linear_regression(regressor, dependent_variable) function takes a vector for the independent variable (as opposed to allowing a matrix of multiple regressors), it's probably not appropriate to use `X` (capitalized). -- ___ Python tracker <https://bugs.python.org/issue44151> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue44151] Improve parameter names and return value ordering for linear_regression
Change by Zachary Kneupper : -- keywords: +patch nosy: +zkneupper nosy_count: 5.0 -> 6.0 pull_requests: +24816 stage: -> patch review pull_request: https://github.com/python/cpython/pull/26199 ___ Python tracker <https://bugs.python.org/issue44151> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue30274] Rename 'name' to 'fullname' argument to ExtensionFileLoader
Change by Zachary Kneupper : -- keywords: +patch nosy: +zkneupper nosy_count: 4.0 -> 5.0 pull_requests: +24812 pull_request: https://github.com/python/cpython/pull/26195 ___ Python tracker <https://bugs.python.org/issue30274> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue44135] issubclass documentation doesn't explain tuple semantic
Change by Zachary Kneupper : -- keywords: +patch nosy: +zkneupper nosy_count: 3.0 -> 4.0 pull_requests: +24810 stage: -> patch review pull_request: https://github.com/python/cpython/pull/26193 ___ Python tracker <https://bugs.python.org/issue44135> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com