Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove btb.tuning.constants module #68

Closed
csala opened this issue May 31, 2018 · 1 comment · Fixed by #148
Closed

Remove btb.tuning.constants module #68

csala opened this issue May 31, 2018 · 1 comment · Fixed by #148
Assignees
@pvk-developer
Labels
good first issue help wanted internal improvement Improvements that do not change the public API
Milestone
0.3.2

Comments

@csala
Copy link
Contributor

csala commented May 31, 2018

The btb.tuning.constants module is not used anywhere, and does not include all the tuners that exist, so it feels it can be removed.
@csala csala added the internal improvement Improvements that do not change the public API label May 31, 2018
@micahjsmith
Copy link
Contributor

Also remove btb.selection.constants

@pvk-developer pvk-developer self-assigned this Dec 3, 2019
@pvk-developer pvk-developer added this to the 0.3.2 milestone Dec 3, 2019
@pvk-developer pvk-developer mentioned this issue Dec 5, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pvk-developer pvk-developer

Labels
good first issue help wanted internal improvement Improvements that do not change the public API
Projects
None yet
Milestone
0.3.2
Development

Successfully merging a pull request may close this issue.

Issue 141 implement propper logging MLBazaar/BTB
3 participants
@micahjsmith @csala @pvk-developer