Skip to content

Commit 877771c

Browse files
committed
Safer way to overwrite timeseries_file
1 parent 2763c39 commit 877771c

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

timeseries.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -258,12 +258,10 @@ def precompute_timeseries (ds_nemo, timeseries_types, timeseries_file, halo=True
258258
ds_new = xr.concat([ds_old, ds_new], dim='time_centered')
259259
ds_old.close()
260260

261-
# Save to file, overwriting if needed
262-
try:
263-
ds_new.to_netcdf(timeseries_file, mode='w')
264-
except(PermissionError):
265-
ds_new.to_netcdf('test.nc', mode='w')
266-
os.rename('test.nc', timeseries_file)
261+
# Save to file
262+
timeseries_file_tmp = timeseries_file.replace('.nc', '_tmp.nc')
263+
ds_new.to_netcdf(timeseries_file_tmp, mode='w')
264+
os.rename(timeseries_file_tmp, timeseries_file)
267265
ds_new.close()
268266

269267

0 commit comments

Comments
 (0)