Don't use uvloop if on Windows and add Windows runner back into CI

golmschenk · Feb 6, 2024 · 38006ad · 38006ad
1 parent b7994a9
commit 38006ad
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/.github/workflows/build_and_test.yaml b/.github/workflows/build_and_test.yaml
@@ -9,7 +9,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ ubuntu-latest, macos-latest ]
+        os: [ ubuntu-latest, windows-latest, macos-latest ]
     steps:
       - uses: actions/checkout@v3
 

diff --git a/src/ramjet/analysis/light_curve_folding_vizualizer/application.py b/src/ramjet/analysis/light_curve_folding_vizualizer/application.py
@@ -1,18 +1,19 @@
+import argparse
+import platform
 from pathlib import Path
 
-import pandas as pd
 import uvloop
 from bokeh import plotting
 from bokeh.application import Application
 from bokeh.application.handlers import DirectoryHandler
 from bokeh.server.server import Server
-import argparse
 
 plotting.output_notebook.__doc__ = ''
 
 
 def run_viewer(light_curve_path: Path, port: int = 5007):
-    uvloop.install()
+    if platform.system() != 'Windows':
+        uvloop.install()
     print(f'Opening viewer on http://localhost:{port}/')
     handler = DirectoryHandler(filename=str(Path(__file__).parent), argv=[light_curve_path])
     application = Application(handler)