Skip to content

Commit

Permalink
Merge pull request #269 from toolsforexperiments/astafan8-inspectr-fix
Browse files Browse the repository at this point in the history
update completed and nrecords columns in inspectr correctly
  • Loading branch information
astafan8 committed Mar 28, 2022
2 parents 8068ab4 + c21255d commit 7c1211e
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions plottr/apps/inspectr.py
Original file line number Diff line number Diff line change
Expand Up @@ -216,12 +216,12 @@ def updateRuns(self, selection: Dict[int, Dict[str, str]]) -> None:
elif len(item) == 1:
completed = record.get('completed_date', '') + ' ' + record.get(
'completed_time', '')
if completed != item[0].text(5):
item[0].setText(5, completed)
if completed != item[0].text(6):
item[0].setText(6, completed)

num_records = str(record.get('records', ''))
if num_records != item[0].text(6):
item[0].setText(6, num_records)
if num_records != item[0].text(7):
item[0].setText(7, num_records)
else:
raise RuntimeError(f"More than one runs found with runId: "
f"{runId}")
Expand Down

0 comments on commit 7c1211e

Please sign in to comment.