Skip to content

Commit

Permalink
minimized diff
Browse files Browse the repository at this point in the history
  • Loading branch information
veenstrajelmer committed Dec 13, 2023
1 parent 6bac0f1 commit e691187
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion dfm_tools/observations.py
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ def ssc_ssh_read_catalog():


def copernicusmarine_retrieve_from_ftp(dataset_id, file_filter, output_directory):
direct = True
direct = False
if direct:
dir_ftp = os.path.dirname(file_filter)
fname = os.path.basename(file_filter)
Expand All @@ -184,7 +184,9 @@ def copernicusmarine_retrieve_from_ftp(dataset_id, file_filter, output_directory
with open(fname_out, 'wb') as fp:
ftp.retrbinary(f'RETR {fname}', fp.write)

Check warning on line 185 in dfm_tools/observations.py

View check run for this annotation

Codecov / codecov/patch

dfm_tools/observations.py#L179-L185

Added lines #L179 - L185 were not covered by tests
else:
# avoid INFO prints from cmc
logging.getLogger("copernicus_marine_root_logger").setLevel("ERROR")

Check warning on line 188 in dfm_tools/observations.py

View check run for this annotation

Codecov / codecov/patch

dfm_tools/observations.py#L188

Added line #L188 was not covered by tests
#TODO: currently still causing "DeprecationWarning: Function 'semver.match' is deprecated."
print() #TODO: temporary newline print to avoid the progressbar messing up the station numbers
cmc.get(

Check warning on line 191 in dfm_tools/observations.py

View check run for this annotation

Codecov / codecov/patch

dfm_tools/observations.py#L190-L191

Added lines #L190 - L191 were not covered by tests
dataset_id=dataset_id,
Expand Down

0 comments on commit e691187

Please sign in to comment.