knakamura13 / mlrose-ky

A highly optimized fork of the popular mlrose-hiive package. For Machine Learning, Randomized Optimization and SEarch algorithms.
https://nkapila6.github.io/mlrose-ky/
BSD 3-Clause "New" or "Revised" License
13 stars 3 forks source link

Support backwards compatibility with mlrose-hiive #7

Open knakamura13 opened 1 month ago

knakamura13 commented 1 month ago

Issue Summary
Refactored code needs to be further revised to ensure backward compatibility with mlrose-hiive.

knakamura13 commented 1 month ago

Progress towards supporting backward compatibility

knakamura13 commented 4 weeks ago

Update

I've decided that the classes, methods, properties, and attributes should be renamed back to their original names for the time being. This revision will inherently support backwards compatibility with mlrose-hiive.

At this time, only the renamed classes have been documented in the comment above, so we'll start with reverting the class names.

The other renamed entities will be more difficult to track down, but they might be just as important as the class names.

knakamura13 commented 2 weeks ago

@gitongah want me to leave the last two class names for you to rename, or you want me to do it?

knakamura13 commented 1 week ago

EXAMPLE COMMENT Finished method names check for .algorithms.decay.

gitongah commented 1 week ago

Finished method name check for .decorators.short_name_decorator.py we have different names from the mlrose-hiive

gitongah commented 1 week ago

Finished method name check for .gridsearch.grid_search_mixin.py we have different name from the mlrose-hiive

gitongah commented 1 week ago

Finished method name check for .runners._nn_runner_base.py and we have a method that is not on the mlrose-hiive. "dynamic_runner_name" method. We also have some method name that are different from mlrose-hiive in .runners._runner_base.py