Update isIpy() to check for interactive notebook

Improves compatibility with Spyder

README.md

@@ -30,7 +30,7 @@ The included [bootstrap model](https://github.com/charwick/helipad/blob/master/s
 
 ## Requirements
 
-Helipad requires Python 3.6 or higher. The following libraries are also required:
+Helipad requires Python 3.7 or higher. The following libraries are also required:
 
 * [Matplotlib](https://matplotlib.org/) (for visualization)
 * [Pandas](https://pandas.pydata.org/) (for data collection)

helipad/helpers.py

@@ -5,11 +5,22 @@
 
 import warnings
 
-def isIpy():
-	try:
-		__IPYTHON__
-		return True
-	except NameError: return False
+#Pass True to check for any Ipython environment, including Spyder, for event loop purposes.
+#Otherwise check specifically whether it's an interactive notebook. However,  get_ipython() comes back
+#undefined inside callbacks. So cache the value once, the first time it runs.
+def isIpy(atall=False):
+	if atall:
+		try:
+			__IPYTHON__
+			return True
+		except NameError: return False
+
+	if not '__helipad_ipy' in globals():
+		try:
+			globals()['__helipad_ipy'] = 'InteractiveShell' in get_ipython().__class__.__name__
+		except NameError: globals()['__helipad_ipy'] = False
+
+	return __helipad_ipy
 
 #Generic extensible item class to store structured data
 class Item:

helipad/model.py

@@ -538,7 +538,7 @@ async def run(self):
 					self.doHooks('visualRefresh', [self, self.visual]) 
 
 				elif getattr(self, 'cpanel', None):
-					if isIpy(): await asyncio.sleep(0.001) #Listen for keyboard input
+					if isIpy(True): await asyncio.sleep(0.001) #Listen for keyboard input
 					else: self.cpanel.parent.update() #Make sure we don't hang the interface if plotless
 
 				# Performance indicator
@@ -567,7 +567,7 @@ def start(self, *args):
 		#that the statement in the try block doesn't get executed…?
 		with warnings.catch_warnings():
 			warnings.simplefilter("ignore")
-			if not isIpy(): asyncio.run(self.run())	#If Tkinter, it needs an event loop
+			if not isIpy(True): asyncio.run(self.run())	#If Tkinter, it needs an event loop
 			else: asyncio.ensure_future(self.run()) #If Jupyter, it already has an event loop
 
 	def stop(self, *args):

setup.py

@@ -34,5 +34,5 @@
 		"Intended Audience :: Science/Research",
 		"Topic :: Scientific/Engineering :: Visualization"
 	],
-	python_requires='>=3.6',
+	python_requires='>=3.7',
 )