Skip to content

Commit 9a1cc86

Browse files
committed
feat: improve job list display
1 parent 2bc561a commit 9a1cc86

5 files changed

Lines changed: 204 additions & 234 deletions

File tree

packages/syft-enclave/src/syft_enclaves/client.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ def jobs(self) -> JobsList:
128128
else j
129129
for j in jobs_list
130130
]
131-
return JobsList(wrapped, jobs_list._root_email)
131+
return JobsList(wrapped, has_do_role=jobs_list._has_do_role)
132132

133133
def submit_python_job(
134134
self,

packages/syft-job/src/syft_job/client.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -662,7 +662,6 @@ def job_sort_key(job):
662662
sorted_jobs = sorted(current_jobs, key=job_sort_key)
663663
return JobsList(
664664
sorted_jobs,
665-
self.target_datasite_owner_email,
666665
has_do_role=self.has_do_role,
667666
)
668667

packages/syft-job/src/syft_job/job.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -426,9 +426,8 @@ def _repr_html_(self) -> str:
426426
class JobsList:
427427
"""A list-like container for JobInfo objects with nice display."""
428428

429-
def __init__(self, jobs: List[JobInfo], root_email: str, has_do_role: bool = False):
429+
def __init__(self, jobs: List[JobInfo], has_do_role: bool = False):
430430
self._jobs = jobs
431-
self._root_email = root_email
432431
self._has_do_role = has_do_role
433432

434433
def __getitem__(self, index: int | str) -> JobInfo:
@@ -449,10 +448,10 @@ def __iter__(self):
449448
return iter(self._jobs)
450449

451450
def __str__(self) -> str:
452-
return jobs_list_str(self._jobs, self._root_email, self._has_do_role)
451+
return jobs_list_str(self._jobs, has_do_role=self._has_do_role)
453452

454453
def __repr__(self) -> str:
455454
return f"JobsList({len(self._jobs)} jobs)"
456455

457456
def _repr_html_(self) -> str:
458-
return jobs_list_repr_html(self._jobs, self._root_email, self._has_do_role)
457+
return jobs_list_repr_html(self._jobs, has_do_role=self._has_do_role)

0 commit comments

Comments
 (0)