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.

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

mxboard should be optional #45

Closed
jwmueller opened this issue Oct 24, 2019 · 4 comments
Closed

mxboard should be optional #45

jwmueller opened this issue Oct 24, 2019 · 4 comments
Labels
API & Doc Improvements or additions to documentation enhancement New feature or request

Comments

@jwmueller
Copy link
Contributor

mxboard should either be optional or it should be explicitly listed as a requirement in setup.py (I vote for the former choice).

It's annoying to keep getting: ModuleNotFoundError: No module named 'mxboard' after import autogluon on new instances because I forgot to install mxboard manually myself.

@cgraywang
Copy link
Contributor

it should already be optional, I think by changing the visualize to none it won't require mxboard.

@jwmueller
Copy link
Contributor Author

jwmueller commented Oct 28, 2019

well then shouldn't visualize set itself automatically based on whether try: import mxboard fails or not?

@cgraywang
Copy link
Contributor

yes, we could do this as well, but we should at the same time raise a warning informing the user that the mxboard is not installed yet

@zhanghang1989 zhanghang1989 added the enhancement New feature or request label Nov 1, 2019
@jwmueller
Copy link
Contributor Author

visualize argument still needs documentation to describe how it can be used, and what (if anything) the user needs to have installed when using visualize

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API & Doc Improvements or additions to documentation enhancement New feature or request
Projects
Development

No branches or pull requests

3 participants