added opotion to pull config from database,

major code refactor

src/cli/data.py

@@ -135,7 +135,8 @@ def set_current_time(client, current_time):
 
 def get_database_config(client):
 
-	return get_analysis_flags(client, "config")["config"]
+	remote_config = get_analysis_flags(client, "config")
+	return remote_config["config"] if remote_config != None else None
 
 def set_database_config(client, config):
 

src/cli/processing.py

@@ -81,12 +81,13 @@ def matchloop(client, competition, data, tests, exec_threads):
 
 	return return_vector
 
-def metricloop(tbakey, client, competition, timestamp, metrics): # listener based metrics update
+def metricloop(client, competition, data, metrics): # listener based metrics update
 
 	elo_N = metrics["elo"]["N"]
 	elo_K = metrics["elo"]["K"]
 
-	matches = pull_new_tba_matches(tbakey, competition, timestamp)
+	matches = data
+	#matches = pull_new_tba_matches(tbakey, competition, timestamp)
 
 	red = {}
 	blu = {}

src/cli/superscript.py

@@ -19,6 +19,10 @@ __changelog__ = """changelog:
 		- added event and time delay options to config
 			- event delay pauses loop until even listener recieves an update
 			- time delay pauses loop until the time specified has elapsed since the BEGINNING of previous loop
+		- added options to pull config information from database (reatins option to use local config file)
+			- config-preference option selects between prioritizing local config and prioritizing database config
+			- synchronize-config option selects whether to update the non prioritized config with the prioritized one
+			- divided config options between persistent ones (keys), and variable ones (everything else)
 	0.9.3:
 		- improved data loading performance by removing redundant PyMongo client creation (120s to 14s)
 		- passed singular instance of PyMongo client as standin for apikey parameter in all data.py functions
@@ -160,18 +164,23 @@ import warnings
 import websockets
 
 from interface import splash, log, ERR, INF, stdout, stderr
-from data import get_previous_time, set_current_time, load_match, push_match, load_pit, push_pit
+from data import get_previous_time, pull_new_tba_matches, set_current_time, load_match, push_match, load_pit, push_pit, get_database_config, set_database_config
 from processing import matchloop, metricloop, pitloop
 
 config_path = "config.json"
 sample_json = """{
-	"max-threads": 0.5,
+	"persistent":{
-	"team": "",
-	"competition": "2020ilch",
 		"key":{
 			"database":"",
 			"tba":""
 		},
+		"config-preference":"local",
+		"synchronize-config":false
+	},
+	"variable":{
+		"max-threads":0.5,
+		"team":"",
+		"competition": "2020ilch",
 		"statistics":{
 			"match":{
 				"balls-blocked":["basic_stats","historical_analysis","regression_linear","regression_logarithmic","regression_exponential","regression_polynomial","regression_sigmoidal"],
@@ -211,14 +220,22 @@ sample_json = """{
 		},
 		"even-delay":false,
 		"loop-delay":60
+	}
 }"""
 
 def main(send, verbose = False, profile = False, debug = False):
 
+	def close_all():
+		if "exec_threads" in locals():
+			exec_threads.terminate()
+			exec_threads.join()
+			exec_threads.close()
+		if "client" in locals():
+			client.close()
+
 	warnings.filterwarnings("ignore")
 	sys.stderr = open("errorlog.log", "w")
-	loop_exit_code = 0
+	exit_code = 0
-	loop_stored_exception = None
 
 	if verbose:
 		splash(__version__)
@@ -229,69 +246,146 @@ def main(send, verbose = False, profile = False, debug = False):
 
 			loop_start = time.time()
 
-			current_time = time.time()
+			send(stdout, INF, "current time: " + str(loop_start))
-			send(stdout, INF, "current time: " + str(current_time))
-
-			send(stdout, INF, "loading config at <" + config_path + ">", code = 0)
 
 			config = {}
-			if load_config(config_path, config) == 1:
+
+			if load_config(config_path, config):
 				send(stderr, ERR, "could not find config at <" + config_path + ">, generating blank config and exiting", code = 100)
-				sys.exit(1)
+				exit_code = 1
+				break
 			
-			send(stdout, INF, "found and opened config at <" + config_path + ">", code = 0)
+			send(stdout, INF, "found and loaded config at <" + config_path + ">")
 
-			error_flag = False
+			flag, apikey, tbakey, preference, sync = parse_config_persistent(send, config)
+			if flag:
+				exit_code = 1
+				break
+			send(stdout, INF, "found and loaded database and tba keys")
 
+			client = pymongo.MongoClient(apikey)
+
+			send(stdout, INF, "established connection to database")
+			send(stdout, INF, "analysis backtimed to: " + str(get_previous_time(client)))
+
+			resolve_config_conflicts(send, client, config, preference, sync)
+			if config == 1:
+				exit_code = 1
+				break
+			flag, exec_threads, competition, match_tests, metrics_tests, pit_tests = parse_config_variable(send, config)
+			if flag:
+				exit_code = 1
+				break
+
+			start = time.time()
+			send(stdout, INF, "loading match, metric, pit data (this may take a few seconds)")
+			match_data = load_match(client, competition)
+			metrics_data = pull_new_tba_matches(tbakey, competition, loop_start)
+			pit_data = load_pit(client, competition)
+			send(stdout, INF, "finished loading match, metric, pit data in "+ str(time.time() - start) + " seconds")
+
+			start = time.time()
+			send(stdout, INF, "performing analysis on match, metrics, pit data")
+			match_results = matchloop(client, competition, match_data, match_tests, exec_threads)
+			metrics_results = metricloop(client, competition, metrics_data, metrics_tests)
+			pit_results = pitloop(client, competition, pit_data, pit_tests)
+			send(stdout, INF, "finished analysis in " + str(time.time() - start) + " seconds")
+
+			start = time.time()
+			send(stdout, INF, "uploading match, metrics, pit results to database")
+			push_match(client, competition, match_results)
+			push_pit(client, competition, pit_results)
+			send(stdout, INF, "finished uploading results in " + str(time.time() - start) + " seconds")
+
+			if debug:
+				f = open("matchloop.log", "w+")
+				json.dump(match_results, f, ensure_ascii=False, indent=4)
+				f.close()
+
+				f = open("pitloop.log", "w+")
+				json.dump(pit_results, f, ensure_ascii=False, indent=4)
+				f.close()
+
+			set_current_time(client, loop_start)
+			close_all()
+
+			send(stdout, INF, "closed threads and database client")
+			send(stdout, INF, "finished all tasks in " + str(time.time() - loop_start) + " seconds, looping")
+
+			if profile:
+				return # return instead of break to avoid sys.exit
+
+			loop_delay = float(config["variable"]["loop-delay"])
+			remaining_time = loop_delay - (time.time() - loop_start)
+			if remaining_time > 0:
+				send(stdout, INF, "loop delayed by " + str(remaining_time) + " seconds")
+				time.sleep(remaining_time)
+
+		except KeyboardInterrupt:
+			send(stdout, INF, "detected KeyboardInterrupt, killing threads")
+			close_all()
+			send(stdout, INF, "terminated threads, exiting")
+			loop_exit_code = 0
+			break
+
+		except Exception as e:
+			send(stderr, ERR, "encountered an exception while running", code = 1)
+			print(e, file = stderr)
+			exit_code = 1
+			close_all()
+			break
+	
+	sys.exit(exit_code)
+
+def parse_config_persistent(send, config):
+
+	exit_flag = False
 	try:
-				competition = config["competition"]
+		apikey = config["persistent"]["key"]["database"]
 	except:
-				send(stderr, ERR, "could not find competition field in config", code = 101)
+		send(stderr, ERR, "database key field in config must be present", code = 111)
-				error_flag = True
+		exit_flag = True
 	try:
-				match_tests = config["statistics"]["match"]
+		tbakey = config["persistent"]["key"]["tba"]
 	except:
-				send(stderr, ERR, "could not find match_tests field in config", code = 102)
+		send(stderr, ERR, "tba key field in config must be present", code = 112)
-				error_flag = True
+		exit_flag = True
 	try:
-				metrics_tests = config["statistics"]["metric"]
+		preference = config["persistent"]["config-preference"]
 	except:
-				send(stderr, ERR, "could not find metrics_tests field in config", code = 103)
+		send(stderr, ERR, "config-preference field in config must be present", code = 113)
-				error_flag = True
+		exit_flag = True
 	try:
-				pit_tests = config["statistics"]["pit"]
+		sync = config["persistent"]["synchronize-config"]
 	except:
-				send(stderr, ERR, "could not find pit_tests field in config", code = 104)
+		send(stderr, ERR, "synchronize-config field in config must be present", code = 114)
-				error_flag = True
+		exit_flag = True
 
-			if error_flag:
+	if apikey == None or apikey == "":
-				sys.exit(1)
+		send(stderr, ERR, "database key field in config must not be empty, please populate the database key", code = 115)
-			error_flag = False
+		exit_flag = True
+	if tbakey == None or tbakey == "":
+		send(stderr, ERR, "tba key field in config must not be empty, please populate the tba key", code = 116)
+		exit_flag = True
+	if preference == None or preference == "":
+		send(stderr, ERR, "config-preference field in config must not be empty, please populate config-preference", code = 117)
+		exit_flag = True
+	if sync != True and sync != False:
+		send(stderr, ERR, "synchronize-config field in config must be a boolean, please populate synchronize-config", code = 118)
+		exit_flag = True
 
-			if competition == None or competition == "":
+	return exit_flag, apikey, tbakey, preference, sync
-				send(stderr, ERR, "competition field in config must not be empty", code = 105)
-				error_flag = True
-			if match_tests == None:
-				send(stderr, ERR, "match_tests field in config must not be empty", code = 106)
-				error_flag = True
-			if metrics_tests == None:
-				send(stderr, ERR, "metrics_tests field in config must not be empty", code = 107)
-				error_flag = True
-			if pit_tests == None:
-				send(stderr, ERR, "pit_tests field in config must not be empty", code = 108)
-				error_flag = True
 
-			if error_flag:
+def parse_config_variable(send, config):
-				sys.exit(1)
 
-			send(stdout, INF, "found and loaded competition, match_tests, metrics_tests, pit_tests from config")
+	exit_flag = False
 
 	sys_max_threads = os.cpu_count()
 	try:
-				cfg_max_threads = config["max-threads"]
+		cfg_max_threads = config["variable"]["max-threads"]
 	except:
 		send(stderr, ERR, "max-threads field in config must not be empty, refer to documentation for configuration options", code = 109)
-				sys.exit(1)
+		exit_flag = True
 
 	if cfg_max_threads > -sys_max_threads and cfg_max_threads < 0 :
 		alloc_processes = sys_max_threads + cfg_max_threads
@@ -303,124 +397,91 @@ def main(send, verbose = False, profile = False, debug = False):
 		alloc_processes = sys_max_threads
 	else:
 		send(stderr, ERR, "max-threads must be between -" + str(sys_max_threads) + " and " + str(sys_max_threads) + ", but got " + cfg_max_threads, code = 110)
-				sys.exit(1)
+		exit_flag = True
 
-			send(stdout, INF, "found and loaded max-threads from config")
-			send(stdout, INF, "attempting to start " + str(alloc_processes) + " threads")
 	try:
 		exec_threads = Pool(processes = alloc_processes)
 	except Exception as e:
 		send(stderr, ERR, "unable to start threads", code = 200)
 		send(stderr, INF, e)
-				sys.exit(1)
+		exit_flag = True
 	send(stdout, INF, "successfully initialized " + str(alloc_processes) + " threads")
 
-			exit_flag = False
-
 	try:
-				apikey = config["key"]["database"]
+		competition = config["variable"]["competition"]
 	except:
-				send(stderr, ERR, "database key field in config must be present", code = 111)
+		send(stderr, ERR, "could not find competition field in config", code = 101)
 		exit_flag = True
 	try:
-				tbakey = config["key"]["tba"]
+		match_tests = config["variable"]["statistics"]["match"]
 	except:
-				send(stderr, ERR, "tba key field in config must be present", code = 112)
+		send(stderr, ERR, "could not find match field in config", code = 102)
+		exit_flag = True
+	try:
+		metrics_tests = config["variable"]["statistics"]["metric"]
+	except:
+		send(stderr, ERR, "could not find metrics field in config", code = 103)
+		exit_flag = True
+	try:
+		pit_tests = config["variable"]["statistics"]["pit"]
+	except:
+		send(stderr, ERR, "could not find pit field in config", code = 104)
 		exit_flag = True
 
-			if apikey == None or apikey == "":
+	if competition == None or competition == "":
-				send(stderr, ERR, "database key field in config must not be empty, please populate the database key")
+		send(stderr, ERR, "competition field in config must not be empty", code = 105)
 		exit_flag = True
-			if tbakey == None or tbakey == "":
+	if match_tests == None:
-				send(stderr, ERR, "tba key field in config must not be empty, please populate the tba key")
+		send(stderr, ERR, "matchfield in config must not be empty", code = 106)
+		exit_flag = True
+	if metrics_tests == None:
+		send(stderr, ERR, "metrics field in config must not be empty", code = 107)
+		exit_flag = True
+	if pit_tests == None:
+		send(stderr, ERR, "pit field in config must not be empty", code = 108)
 		exit_flag = True
 
-			if exit_flag:
+	send(stdout, INF, "found and loaded competition, match, metrics, pit from config")
-				sys.exit(1)
 
-			send(stdout, INF, "found and loaded database and tba keys")
+	return exit_flag, exec_threads, competition, match_tests, metrics_tests, pit_tests
 
-			client = pymongo.MongoClient(apikey)
+def resolve_config_conflicts(send, client, config, preference, sync):
 
-			previous_time = get_previous_time(client)
+	if sync:
-			send(stdout, INF, "analysis backtimed to: " + str(previous_time))
+		if preference == "local" or preference == "client":
-
+			send(stdout, INF, "config-preference set to local/client, loading local config information")
-			start = time.time()
+			remote_config = get_database_config(client)
-			send(stdout, INF, "loading match data")
+			if remote_config != config["variable"]:
-			match_data = load_match(client, competition)
+				set_database_config(client, config["variable"])
-			send(stdout, INF, "finished loading match data in " + str(time.time() - start) + " seconds")
+				send(stdout, INF, "database config was different and was updated")
-
+			return
-			start = time.time()
+		elif preference == "remote" or preference == "database":
-			send(stdout, INF, "performing analysis on match data")
+			send(stdout, INF, "config-preference set to remote/database, loading remote config information")
-			results = matchloop(client, competition, match_data, match_tests, exec_threads)
+			remote_config= get_database_config(client)
-			send(stdout, INF, "finished match analysis in " + str(time.time() - start) + " seconds")
+			if remote_config != config["variable"]:
-
+				config["variable"] = remote_config
-			if debug:
+				if save_config(config_path, config):
-				f = open("matchloop.log", "w+")
+					send(stderr, ERR, "local config was different but could not be updated")
-				json.dump(results, f)
+					config = 1
-				f.close()
+					return
-
+				send(stdout, INF, "local config was different and was updated")
-			start = time.time()
+			return
-			send(stdout, INF, "uploading match results to database")
+		else:
-			push_match(client, competition, results)
+			send(stderr, ERR, "config-preference field in config must be \"local\"/\"client\" or \"remote\"/\"database\"")
-			send(stdout, INF, "finished uploading match results in " + str(time.time() - start) + " seconds")
+			config = 1
-
+			return
-			start = time.time()
+	else:
-			send(stdout, INF, "performing analysis on team metrics")
+		if preference == "local" or preference == "client":
-			results = metricloop(tbakey, client, competition, current_time, metrics_tests)
+			send(stdout, INF, "config-preference set to local/client, loading local config information")
-			send(stdout, INF, "finished metric analysis and pushed to database in " + str(time.time() - start) + " seconds")
+			return
-
+		elif preference == "remote" or preference == "database":
-			start = time.time()
+			send(stdout, INF, "config-preference set to remote/database, loading database config information")
-			send(stdout, INF, "loading pit data")
+			config["variable"] = get_database_config(client)
-			pit_data = load_pit(client, competition)
+			return
-			send(stdout, INF, "finished loading pit data in " + str(time.time() - start) + " seconds")
+		else:
-
+			send(stderr, ERR, "config-preference field in config must be \"local\"/\"client\" or \"remote\"/\"database\"")
-			start = time.time()
+			config = 1
-			send(stdout, INF, "performing analysis on pit data")
-			results = pitloop(client, competition, pit_data, pit_tests)
-			send(stdout, INF, "finished pit analysis in " + str(time.time() - start) + " seconds")
-
-			if debug:
-				f = open("pitloop.log", "w+")
-				json.dump(results, f)
-				f.close()
-
-			start = time.time()
-			send(stdout, INF, "uploading pit results to database")
-			push_pit(client, competition, results)
-			send(stdout, INF, "finished uploading pit results in " + str(time.time() - start) + " seconds")
-
-			client.close()
-
-			set_current_time(client, current_time)
-			send(stdout, INF, "finished all tests in " + str(time.time() - loop_start) + " seconds, looping")
-
-			loop_delay = float(config["loop-delay"])
-			remaining_time = loop_delay - (time.time() - loop_start)
-			if remaining_time > 0:
-				send(stdout, INF, "loop delayed by " + str(remaining_time) + " seconds")
-				time.sleep(remaining_time)
-
-		except KeyboardInterrupt:
-			send(stdout, INF, "detected KeyboardInterrupt, killing threads")
-			if "exec_threads" in locals():
-				exec_threads.terminate()
-				exec_threads.join()
-				exec_threads.close()
-			send(stdout, INF, "terminated threads, exiting")
-			loop_stored_exception = sys.exc_info()
-			loop_exit_code = 0
-			break
-		except Exception as e:
-			send(stderr, ERR, "encountered an exception while running", code = 1)
-			print(e, file = stderr)
-			loop_exit_code = 1
-			break
-
-		if profile:
 			return
-
-	sys.exit(loop_exit_code)
 
 def load_config(path, config_vector):
 	try:
@@ -435,13 +496,10 @@ def load_config(path, config_vector):
 		return 1
 
 def save_config(path, config_vector):
-	try:
+	f = open(path, "w+")
-		f = open(path)
+	json.dump(config_vector, f, ensure_ascii=False, indent=4)
-		json.dump(config_vector)
 	f.close()
 	return 0
-	except:
-		return 1
 
 def start(pid_path, verbose = False, profile = False, debug = False):