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

Add basic CLI #97

Closed
wants to merge 6 commits into from
Closed

Add basic CLI #97

wants to merge 6 commits into from

Conversation

indyfree
Copy link
Collaborator

@indyfree indyfree commented Aug 1, 2021

Towards #96

@indyfree indyfree self-assigned this Aug 1, 2021
@@ -4,6 +4,10 @@

from ._logger import INFOFORMATTER

import warnings

warnings.filterwarnings("ignore", message=r"Passing", category=FutureWarning)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI @Philoso-Fish , this one supresses the ugly:

/home/morty/dev/carla/env/lib/python3.7/site-packages/tensorflow/python/framework/dtypes.py:516: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_qint8 = np.dtype([("qint8", np.int8, 1)])
/home/morty/dev/carla/env/lib/python3.7/site-packages/tensorflow/python/framework/dtypes.py:517: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.

warnings

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is really great 👍

@indyfree indyfree changed the title Add basic cli skaffold WIP: Add basic CLI Aug 1, 2021
@indyfree indyfree marked this pull request as draft August 1, 2021 19:55
Base automatically changed from feature/add-pypi-functionality to main August 18, 2021 13:36
@indyfree indyfree changed the title WIP: Add basic CLI Add basic CLI Aug 29, 2021
@indyfree indyfree force-pushed the feature/add-cli-functionality branch from 5785ce1 to 7e3be25 Compare August 29, 2021 18:04
import tensorflow as tf
import torch


tf.get_logger().setLevel(logging.ERROR)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove more tensorflow warnings

"Click==8.0.1",
"dice-ml==0.5",
"h5py==2.10.0",
"ipython",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Philoso-Fish do we really need ipyhon btw? That seems to be more for notebooks etc?

@indyfree indyfree marked this pull request as ready for review August 29, 2021 18:06
@indyfree indyfree closed this Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants