Skip to content

Commit

Permalink
Remove pretty_print from Versioner
Browse files Browse the repository at this point in the history
Including a printer for Versioner would require tabulate as an
additional dependency. For now, this functionality has been
removed to avoid errors when installing via pip.
  • Loading branch information
ashao committed Apr 2, 2022
1 parent 1321846 commit 7f9cff0
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 11 deletions.
2 changes: 0 additions & 2 deletions smartsim/_core/_cli/build.py
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,6 @@ def __init__(self):
logger.error("SMARTSIM_DEP_PATH not set. Please contact your site administrator")

self.versions=Versioner()
logger.info("Version Information:")
self.versions.pretty_print()

self.install_torch(smartsim_dependency_path)

Expand Down
4 changes: 0 additions & 4 deletions smartsim/_core/_cli/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ def _usage():
"\nDeveloper:",
"\tsite Print the installation site of SmartSim",
"\tdbcli Print the path to the redis-cli binary" "\n\n",
"\tversions Display versions of SmartSim and related packages"
]
return "\n".join(usage)

Expand Down Expand Up @@ -70,9 +69,6 @@ def dbcli(self):
print("Redis dependencies not found")
exit(1)

def versions(self):
Versioner().pretty_print()

def sitebuild(self):
SiteBuild()

Expand Down
5 changes: 0 additions & 5 deletions smartsim/_core/_install/buildenv.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
import subprocess
import sys
from pathlib import Path
from tabulate import tabulate
from typing import Iterable

import pkg_resources
Expand Down Expand Up @@ -211,10 +210,6 @@ class Versioner:
TENSORFLOW = Version_(REDISAI.tensorflow)
ONNX = Version_(REDISAI.onnx)

def pretty_print(self):
vers = self.as_dict()
print(tabulate(vers, headers=vers.keys(), tablefmt="github"), "\n")

def as_dict(self):
packages = [
"SMARTSIM",
Expand Down

0 comments on commit 7f9cff0

Please sign in to comment.