Updated print commands

wellsoliver · Nov 16, 2021 · dac9a71 · dac9a71
1 parent 8e4a9fa
commit dac9a71
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 20 deletions.
diff --git a/scripts/classes/fetcher.py b/scripts/classes/fetcher.py
@@ -29,7 +29,7 @@ def run(self):
 
             # log
             if(self.options['verbose']):
-                print "Fetching " + filename
+                print ("Fetching " + filename)
 
             # determine the local path
             f = "%s/%s" % (self.path, filename)
@@ -42,7 +42,7 @@ def run(self):
 
                 #log
                 if(self.options['verbose']):
-                    print "Zip file detected. Extracting " + filename
+                    print("Zip file detected. Extracting " + filename)
 
                 # extract the zip file
                 zip = zipfile.ZipFile(f, "r")

diff --git a/scripts/download.py b/scripts/download.py
@@ -31,7 +31,7 @@
 try:
     os.chdir(absolute_path)
 except OSError:
-    print "Directory %s does not exist, creating..." % absolute_path
+    print ("Directory %s does not exist, creating..." % absolute_path)
     os.makedirs(absolute_path)
 
 
@@ -40,7 +40,7 @@
 try:
     opts, args = getopt.getopt(sys.argv[1:], "y:", ["year="])
 except getopt.GetoptError as e:
-    print 'Invalid arguments. Exiting.'
+    print ('Invalid arguments. Exiting.')
     raise SystemExit
 
 # set year if passed in
@@ -55,9 +55,9 @@
 
     # log next action
     if YEAR:
-        print "Queuing up Event Files for download (%s only)." % YEAR
+        print ("Queuing up Event Files for download (%s only)." % YEAR)
     else:
-        print "Queuing up Event Files for download."
+        print ("Queuing up Event Files for download.")
 
     # parse retrosheet page for files and add urls to the queue
     retrosheet_url = config.get('retrosheet', 'eventfiles_url')
@@ -82,9 +82,9 @@
 
     # log next action
     if YEAR:
-        print "Queuing up Game Logs for download (%s only)." % YEAR
+        print ("Queuing up Game Logs for download (%s only)." % YEAR)
     else:
-        print "Queuing up Game Logs for download."
+        print ("Queuing up Game Logs for download.")
 
     # parse retrosheet page for files and add urls to the queue
     retrosheet_url = config.get('retrosheet', 'gamelogs_url')

diff --git a/scripts/parse.py b/scripts/parse.py
@@ -21,8 +21,7 @@ def connect(config):
         USER = None if not config.has_option('database', 'user') else config.get('database', 'user')
         SCHEMA = None if not config.has_option('database', 'schema') else config.get('database', 'schema')
         PASSWORD = None if not config.has_option('database', 'password') else config.get('database', 'password')
-    except ConfigParser.NoOptionError:
-        print 'Need to define engine, user, password, host, and database parameters'
+        print ('Need to define engine, user, password, host, and database parameters')
         raise SystemExit
 
     if ENGINE == 'sqlite':
@@ -42,12 +41,12 @@ def connect(config):
 
 
 def parse_rosters(file, conn, bound_param):
-    print "processing %s" % file
+    print ("processing %s" % file)
 
     try:
         year = re.search(r"\d{4}", os.path.basename(file)).group(0)
     except:
-        print 'cannot get year from roster file %s' % file
+        print ('cannot get year from roster file %s' % file)
         return None
 
     reader = csv.reader(open(file))
@@ -68,7 +67,7 @@ def parse_rosters(file, conn, bound_param):
 
 
 def parse_teams(file, conn, bound_param):
-    print "processing %s" % file
+    print ("processing %s" % file)
 
     reader = csv.reader(open(file))
     for row in reader:
@@ -86,12 +85,12 @@ def parse_teams(file, conn, bound_param):
 
 
 def parse_games(file, conn, bound_param):
-    print "processing %s" % file
+    print ("processing %s" % file)
 
     try:
         year = re.search(r"\d{4}", os.path.basename(file)).group(0)
     except:
-        print 'cannot get year from game file %s' % file
+        print ('cannot get year from game file %s' % file)
         return None
 
     if conn.engine.driver == 'psycopg2':
@@ -112,12 +111,12 @@ def parse_games(file, conn, bound_param):
 
 
 def parse_events(file, conn, bound_param):
-    print "processing %s" % file
+    print ("processing %s" % file)
 
     try:
         year = re.search(r"\d{4}", os.path.basename(file)).group(0)
     except:
-        print 'cannot get year from event file %s' % file
+        print ('cannot get year from event file %s' % file)
         return None
 
     if conn.engine.driver == 'psycopg2':
@@ -185,7 +184,7 @@ def main():
     if not os.path.exists(chadwick) \
         or not os.path.exists('%s/cwevent' % chadwick) \
         or not os.path.exists('%s/cwgame' % chadwick):
-        print 'chadwick does not exist in %s - exiting' % chadwick
+        print ('chadwick does not exist in %s - exiting' % chadwick)
         raise SystemExit
 
     os.chdir(path) # Chadwick seems to need to be in the directory
@@ -213,13 +212,13 @@ def main():
         if not os.path.isfile('%s/events-%d.csv' % (csvpath, year)):
             cmd = "%s/cwevent -q -n -f 0-96 -x 0-62 -y %d %d*.EV* > %s/events-%d.csv" % (chadwick, year, year, csvpath, year)
             if(verbose):
-                print "calling '" + cmd + "'"
+                print ("calling '" + cmd + "'")
             subprocess.call(cmd, shell=True)
 
         if not os.path.isfile('%s/games-%d.csv' % (csvpath, year)):
             cmd = "%s/cwgame -q -n -f 0-83 -y %d %d*.EV* > %s/games-%d.csv" % (chadwick, year, year, csvpath, year)
             if(verbose):
-                print "calling '" + cmd + "'"
+                print ("calling '" + cmd + "'")
             subprocess.call(cmd, shell=True)
 
     if 'teams' in modules: