Skip to content

396 convert regressors to num power#411

Open
apphp wants to merge 152 commits into
RubixML:3.0from
apphp:396-convert-regressors-to-num-power
Open

396 convert regressors to num power#411
apphp wants to merge 152 commits into
RubixML:3.0from
apphp:396-convert-regressors-to-num-power

Conversation

@apphp

@apphp apphp commented Mar 28, 2026

Copy link
Copy Markdown

No description provided.

@apphp apphp self-assigned this Mar 28, 2026
* @author Andrew DalPino
* @author Samuel Akopyan <leumas.a@gmail.com>
*/
class KDNeighborsRegressor implements Estimator, Learner, Persistable

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's been a while ... can you explain to me again why we need to create these namespaces?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

* @author Andrew DalPino
* @author Samuel Akopyan <leumas.a@gmail.com>
*/
class KNNRegressor implements Estimator, Learner, Online, Persistable

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again, let's not power this model with NumPower yet.

@@ -0,0 +1,232 @@
<?php

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert back to no NumPower.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted

@@ -0,0 +1,203 @@
<?php

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like it's just a copy of the original file just in a different namespace.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted

apphp added 26 commits June 23, 2026 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants