updated windows support

pepkit · Jan 18, 2024 · 0cbb0c2 · 0cbb0c2
1 parent 256f5f5
commit 0cbb0c2
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 42 deletions.
diff --git a/.github/workflows/run-pytest-windows.yml b/.github/workflows/run-pytest-windows.yml
diff --git a/docs/README.md b/docs/README.md
@@ -54,6 +54,10 @@ geofetch -i GSE95654 --just-metadata
 geofetch -i GSE95654 --processed --just-metadata
 ```
 
+
+&#8291;**Note:** We ensure that GEOfetch is compatible with Unix, Linux, and Mac OS X. 
+However, due to dependencies, some features of GEOfetch may not be available on Windows.
+
 ### Check out what exactly argument you want to use to download data:
 
 ![](./img/arguments_outputs.svg)

diff --git a/docs/changelog.md b/docs/changelog.md
@@ -1,5 +1,8 @@
 # Changelog
 
+## [0.12.6] -- 2024-01-18
+- Updated support for Windows (Some of the functionality could not be available on Windows)
+
 ## [0.12.5] -- 2023-11-29
 - Fixed bug, where description was not populated in PEP
 

diff --git a/geofetch/_version.py b/geofetch/_version.py
@@ -1 +1 @@
-__version__ = "0.12.5"
+__version__ = "0.12.6"
diff --git a/geofetch/geofetch.py b/geofetch/geofetch.py
@@ -1796,15 +1796,22 @@ def _download_processed_file(self, file_url: str, data_folder: str) -> bool:
                 return True
 
             except IOError as e:
-                _LOGGER.error(str(e))
-                # The server times out if we are hitting it too frequently,
-                # so we should sleep a bit to reduce frequency
-                sleeptime = (ntry + 1) ** 3
-                _LOGGER.info(f"Sleeping for {sleeptime} seconds")
-                time.sleep(sleeptime)
-                ntry += 1
-                if ntry > 4:
-                    raise e
+                if os.name == "nt":
+                    _LOGGER.error(f"{e}")
+                    raise OSError(
+                        "Windows may not have wget command. "
+                        "Check if `wget` command is installed correctly."
+                    )
+                else:
+                    _LOGGER.error(str(e))
+                    # The server times out if we are hitting it too frequently,
+                    # so we should sleep a bit to reduce frequency
+                    sleeptime = (ntry + 1) ** 3
+                    _LOGGER.info(f"Sleeping for {sleeptime} seconds")
+                    time.sleep(sleeptime)
+                    ntry += 1
+                    if ntry > 4:
+                        raise e
 
     def _get_SRA_meta(self, file_gse_content: list, gsm_metadata, file_sra=None):
         """