From 79a1a5f2241ee0352be738e2ede25d88e9d533d5 Mon Sep 17 00:00:00 2001 From: Jessica Scheick Date: Thu, 4 Jan 2024 01:52:53 +0000 Subject: [PATCH] put warning back since it depends on what triggers the confirm proceed step --- icepyx/core/read.py | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/icepyx/core/read.py b/icepyx/core/read.py index df13434fe..f2dae6495 100644 --- a/icepyx/core/read.py +++ b/icepyx/core/read.py @@ -266,13 +266,6 @@ def _confirm_proceed(): Ask the user if they wish to proceed with processing. If 'y', or 'yes', then continue. Any other user input will abort the process. """ - warnings.warn( - "Processing more than two s3 files can take a prohibitively long time. " - "Approximate access time (using `.load()`) can exceed 6 minutes per data " - "variable.", - stacklevel=2, - ) - answer = input("Do you wish to proceed (not recommended) y/[n]?") if answer.lower() in ["y", "yes"]: pass @@ -442,6 +435,12 @@ def __init__( self.is_s3 = self.is_s3[0] # Change is_s3 into one boolean value for _filelist # Raise warning if more than 2 s3 files are given if self.is_s3 is True and len(self._filelist) > 2: + warnings.warn( + "Processing more than two s3 files can take a prohibitively long time. " + "Approximate access time (using `.load()`) can exceed 6 minutes per data " + "variable.", + stacklevel=2, + ) _confirm_proceed() # Raise warnings or errors for multiple products or products not matching the user-specified product