Remove profiling code in favor of -m cProfile

gwax · May 28, 2017 · 23d80ad · 23d80ad
1 parent 70eec25
commit 23d80ad
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 43 deletions.
diff --git a/README.rst b/README.rst
@@ -152,6 +152,12 @@ Changelog
 master
 ------
 
+-   Remove profiling code. If we care, we can invoke profiling with:
+
+    .. code:: sh
+
+        python -m cProfile -o mtg_ssm.prof mtg_ssm/ssm.py create collection.xlsx
+
 -   Fix wheel generation to only build py3 wheels.
 
 1.3.4

diff --git a/mtg_ssm/profiling.py b/mtg_ssm/profiling.py
diff --git a/mtg_ssm/ssm.py b/mtg_ssm/ssm.py
@@ -10,7 +10,6 @@
 import mtg_ssm
 
 from mtg_ssm import mtgjson
-from mtg_ssm import profiling
 import mtg_ssm.serialization.interface as ser_interface
 from mtg_ssm.mtg import card_db
 from mtg_ssm.mtg import counts
@@ -45,9 +44,6 @@ def get_args(args=None):
     parser.add_argument(
         '--include-online-only', default=False, action='store_true',
         help='Include online only sets (e.g. Masters sets)')
-    parser.add_argument(
-        '--profile-stats', default=False, action='store_true',
-        help='Output profiling statistics (for debugging)')
 
     parser.add_argument(
         '-d', '--dialect', nargs=2, metavar=('EXTENSION', 'DIALECT'),
@@ -197,8 +193,7 @@ def diff_cmd(args):
 def main():
     """Get args and run the appropriate command."""
     args = get_args()
-    with profiling.profiled(enabled=args.profile_stats):
-        args.func(args)
+    args.func(args)
 
 
 if __name__ == '__main__':