From d1ba34a251a8006227e8d94163f4f14f3e324f8d Mon Sep 17 00:00:00 2001
From: Thomas-George-T <tgt555@gmail.com>
Date: Wed, 18 Oct 2023 13:51:56 -0400
Subject: [PATCH] Adding pylint

---
 src/__pycache__/datapipeline.cpython-310.pyc | Bin 1124 -> 1264 bytes
 src/datapipeline.py                          |  40 +++++++++++--------
 2 files changed, 24 insertions(+), 16 deletions(-)

diff --git a/src/__pycache__/datapipeline.cpython-310.pyc b/src/__pycache__/datapipeline.cpython-310.pyc
index e380b829671ccf225c0ccfdb7b5f343c612a9466..7553f1ba61c0a18aff13bbe3e7831f2ad1aa5d9b 100644
GIT binary patch
delta 803
zcmY+BL2DC16vyYyOtRZ0S!>P3Sj&o{jfhdigCHUmiU$d35NQyWHE*h0lie`8As7;K
zh$lg$?y)F(^BedT{0w{c(wpGTgZ0gZYCFq&GxO%{y!Zdj5Bukm8&|6iqw%YLO?{QS
z?&|C=?T~dC0`eYopqP1wqd@tFbp)2MOtVxav|e((ro@~9Q_^_k?ZO)N_@c8pjJjzU
zM~RolUKl-BN$Ld=dIKGIRg!oZq=B<<inqGPFUIcl#7xr?Vaigr0~5Ls#XCNQ3G8r`
zRI-^CP0aawcgiRHv!s*K7XbEzp(UAQa{(-}ue0RBh@$4(B1_9?&uNcV+pL$&eS7R9
zKa4e;@fF9Tap|yg(>TVXs22z5ZHGPOZR@!2J%03{$jbmW*4L=l2IN*6_EkJg_rKHr
zchzWX8h3RZ4st$v*2+5Y6!PpF+=t~4{NXK&@{(n8Qo6zs74E|7L50`2UZLO-AdCqL
z*kt#}lGoW=zCM%3WS%)Dli*C)9>ekkNS-wE2#jnK7y>F^-B}<Zh?dD)0N5%rMvPvr
zS%^EzU>K1KFMXj7X`Drci}y3Ubcq@pmkP}}=vH<^+{}97+)9<=xw8>qCmd`NzJ|y?
ziSsjUWG~#wBBQw9H-PK3Pio|(l)%A-3Q?&On;P%~ecEK(zTXeR$oCuN-12>lyS}f@
wFWMk2=>1EKg1(~D|Mkr3vM#FHxBD>;dx~&SO#@vrTi9T8`;_gL!2$W_FOAo(=>Px#

delta 611
zcmYjNQEL-H5T2RcyKK+oG+-@A1dSEE2MIn1wiE?LP=tcyK^lZ3WKSARF5&J<>~Uuc
z@m<RLAQYkf2mSzmfWIM+;t%MfPsZ5<tGmq1&dfLa&CJ=#PvwtlH4peapSPkP;>d42
z+G0<G#sPGe7CWd(ix1EtEw#f|L{htNDQQGB1%yagt+Bn3pmOy2`ax`e!4fh!!2(XP
z!~)-tCa$`Z2`#jo(nr6ff=(o7oKr-2$$XOUR3OCaE~L!~NA2BV$z1JB+1JpvZ`b~J
z3}5JPEU0Yf4|lU{oNl&SW*6_9uefHu<16f#Tl5rbCZ-n~lKV<8v6AEoPgKdr#=J_G
zDY?kO9?SW$P`jJQEa@lni*6&B8{%3|7H|yODUj)<rMHc@x$#WyKSL<F{k+aYx(i&T
z5dUyHj4Z`_Ve!k?1Sb%v?L;t3?&xmTZOt$--^4wU)22BS&AM%3w({g%FB)h4Q5;CK
zE^i*pFiL8iSt{RRyuCQ@kHgpfLA1(#dWDrliG)5@dCsdmZ4=cY&672YS7A8p_Tw-N
yoW$WkQn$)3;yfRBhfx^*N3@`1wskJw8jkedAbNmYu~@1Cg^E;NU2xGu`TGwsUU}jG

diff --git a/src/datapipeline.py b/src/datapipeline.py
index 5073ab1..1418250 100644
--- a/src/datapipeline.py
+++ b/src/datapipeline.py
@@ -1,34 +1,42 @@
-import requests
+"""
+Functions to ingest and process data
+"""
 import zipfile
+import requests
 
 def ingest_data():
-  
-  file_url = "https://archive.ics.uci.edu/static/public/352/online+retail.zip"
+    """
+    Function to download file from URL
+    """
+    file_url = "https://archive.ics.uci.edu/static/public/352/online+retail.zip"
 
-  # Send an HTTP GET request to the URL
-  response = requests.get(file_url)
+    # Send an HTTP GET request to the URL
+    response = requests.get(file_url, timeout=30)
 
-  # Check if the request was successful (status code 200)
-  if response.status_code == 200:
-      # Save file to data
-      with open("data/data.zip", "wb") as file:
-          file.write(response.content)
-      print("File downloaded successfully.")
-  else:
-      print(f"Failed to download the file. Status code: {response.status_code}")
+    # Check if the request was successful (status code 200)
+    if response.status_code == 200:
+        # Save file to data
+        with open("data/data.zip", "wb") as file:
+            file.write(response.content)
+        print("File downloaded successfully.")
+    else:
+        print(f"Failed to download the file. Status code: {response.status_code}")
 
 
 def unzip_file():
+    """
+    Function to unzip the downloaded data
+    """
     zip_filename ='data/data.zip'
     extract_to = 'data/'
     try:
         with zipfile.ZipFile(zip_filename, 'r') as zip_ref:
             zip_ref.extractall(extract_to)
         print(f"File {zip_filename} successfully unzipped to {extract_to}")
-    except Exception as e:
-        print(f"Failed to unzip {zip_filename}: {e}")
+    except zipfile.BadZipFile:
+        print(f"Failed to unzip {zip_filename}")
 
 
 if __name__ == "__main__":
     ingest_data()
-    unzip_file()
\ No newline at end of file
+    unzip_file()