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

Noisy warning while using the DeepHyper CLI #24

Closed
Deathn0t opened this issue Dec 11, 2019 · 2 comments
Closed

Noisy warning while using the DeepHyper CLI #24

Deathn0t opened this issue Dec 11, 2019 · 2 comments
Labels
enhancement New feature or request warning Warning when using the package

Comments

@Deathn0t
Copy link
Member

Deathn0t commented Dec 11, 2019

I quote @felker from #23

Still, curious that DeepHyper loads TensorFlow (and its noisy warnings) even for the --help dialog.

The problem is happening when using:

$ deephyper --help

Indeed these warnings are flooding the shell and are very annoying, I create this issue so that we keep thinking about solving this.

@Deathn0t Deathn0t added enhancement New feature or request warning Warning when using the package labels Dec 11, 2019
@felker
Copy link
Member

felker commented Dec 11, 2019

For my project, I just suppress them in __init__.py wiht:

warnings.filterwarnings('ignore',
                        category=FutureWarning,
                        message=r"passing \(type, 1\) or '1type' as a synonym of type is deprecated",
                        module="tensor*")

They come from TensorFlow with NumPy >= v1.17.0.

But it is probably better to bypass importing TensorFlow entirely when --help is used, if possible.

@Deathn0t
Copy link
Member Author

To build the DeepHyper CLI parser we import the search classes and do some introspection on them. So the PPO search using reinforcement learning is imported and it itself imports Tensorflow. I recently noticed that a Mpi4py mock was added to this process to avoid mpi errors while using the CLI when openmpi is not installed (e.g., Theta login nodes). Thus, the deephyper nas ppo.. was not working at all. Given this two issues, I decided to remove the PPO search from the CLI. Tensorflow is not loaded anymore while using the parser.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request warning Warning when using the package
Projects
None yet
Development

No branches or pull requests

2 participants