Skip to content

Commit

Permalink
Use properties for files_to_remove/compress in JobCore.compress
Browse files Browse the repository at this point in the history
Also fixes a typo in to_dict that overwrote files_to_compress with
files_to_remove.
  • Loading branch information
pmrv committed Oct 28, 2024
1 parent 6ff1b2f commit 3fff607
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions pyiron_base/jobs/job/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -1158,12 +1158,12 @@ def compress(
Args:
files_to_compress (list):
"""
if files_to_compress is None and len(self._files_to_compress) != 0:
files_to_compress = self._files_to_compress
if files_to_compress is None and len(self.files_to_compress) != 0:
files_to_compress = self.files_to_compress
elif files_to_compress is None:
files_to_compress = self.files.list()
if files_to_remove is None:
files_to_remove = self._files_to_remove
files_to_remove = self.files_to_remove
else:
files_to_remove = []
_job_compress(
Expand Down
6 changes: 3 additions & 3 deletions pyiron_base/jobs/job/generic.py
Original file line number Diff line number Diff line change
Expand Up @@ -1219,10 +1219,10 @@ def _to_dict(self) -> dict:
data_dict["import_directory"] = self._import_directory
if self._executor_type is not None:
data_dict["executor_type"] = self._executor_type
if len(self._files_to_compress) > 0:
data_dict["files_to_compress"] = self._files_to_compress
if len(self.files_to_compress) > 0:
data_dict["files_to_compress"] = self.files_to_compress
if len(self._files_to_remove) > 0:
data_dict["files_to_compress"] = self._files_to_remove
data_dict["files_to_remove"] = self._files_to_remove
data_dict["HDF_VERSION"] = self.__version__
return data_dict

Expand Down

0 comments on commit 3fff607

Please sign in to comment.