Skip to content
Snippets Groups Projects

Misc fixes from issues

Merged Misc fixes from issues
misc-small-issues into master
1 unresolved thread
Merged Simon Bjurek requested to merge misc-small-issues into master
1 unresolved thread

  • Fixed bug when calculating deadlines for algorithms where the ALAP schedule contained laps.

  • Replaced all startswith checks with isinstance for operations.

  • Updated SchedulingGUI to show "∞" instead of maximum integer for forward and backward slack.

  • Added enforcing of execution time in ListScheduler and added test to check that latency-offsets and execution times will raise Exceptions if missing.

  • Updated ALAP scheduler to adjust scheduling time if a "gap" appears in the start of the schedule.

Closes: #324 (closed) #322 (closed) #318 (closed) #277 (closed) #327 (closed)

Edited by Simon Bjurek

Merge request reports

Pipeline #158882 passed

Stage: test

Pipeline passed for c07be8d8 on misc-small-issues

Test coverage 79.00% (0.00%) from 8 jobs
Approval is optional
Ready to merge by members who can write to the target branch.

Merge details

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
b_asic/scheduler_gui/main_window.py
922 else:
923 self.info_table.setItem(si, 1, QTableWidgetItem("∞"))
922 924 si += 1
923 925
924 926 self.info_table.insertRow(si)
925 927 self.info_table.setItem(si, 0, QTableWidgetItem("Backward slack"))
926 self.info_table.setItem(
927 si,
928 1,
929 QTableWidgetItem(str(self.schedule.backward_slack(graph_id))),
930 )
928 backward_slack = self.schedule.backward_slack(graph_id)
929 if backward_slack < sys.maxsize:
930 self.info_table.setItem(si, 1, QTableWidgetItem(str(backward_slack)))
931 else:
932 self.info_table.setItem(si, 1, QTableWidgetItem("∞"))
  • Oscar Gustafsson approved this merge request

    approved this merge request

  • Oscar Gustafsson merged

    merged

    • Please register or sign in to reply