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

Compartmentalize runtime dependencies on mlrose_hiive #1155

Closed
lbianchi-lbl opened this issue Jul 25, 2023 · 0 comments · Fixed by #1157
Closed

Compartmentalize runtime dependencies on mlrose_hiive #1155

lbianchi-lbl opened this issue Jul 25, 2023 · 0 comments · Fixed by #1157
Assignees
@sotorrio1 @lbianchi-lbl
Labels
Priority:Normal Normal Priority Issue or PR
Milestone
EY23.4.1.D: Refac...

Comments

@lbianchi-lbl
Copy link
Contributor

mlrose_hiive has caused a number of problems, most recently #1154. Since the functionality that relies on it is limited (potentially to a single function?), we should try to move the imports so that potential future errors caused by it are as contained as possible.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sotorrio1 sotorrio1

@lbianchi-lbl lbianchi-lbl

Labels
Priority:Normal Normal Priority Issue or PR
Projects
None yet
Milestone
EY23.4.1.D: Refactoring FOQUS plugin ...
Development

Successfully merging a pull request may close this issue.

Simpler implementation of ordering algorithm sotorrio1/FOQUS
3 participants
@ksbeattie @sotorrio1 @lbianchi-lbl