Skip to content

Workflow and device communication: Robustness and performance update - #199

Merged
chdinh merged 58 commits into
mainfrom
dagster-assets
Feb 1, 2026
Merged

Workflow and device communication: Robustness and performance update#199
chdinh merged 58 commits into
mainfrom
dagster-assets

Conversation

@schote

@schote schote commented Oct 7, 2025

Copy link
Copy Markdown
Collaborator

Improved dagster workflow implementation to increase performance:

  • Define reconstruction job as asset
  • Define ISMRMRD data as asset
  • Implement IO manager for mrpro IData (save/load DICOM result)
  • Implement sensors which listen to run status

User Interface:

  • Make device parameters adjustable
  • Fix device status in status bar (sometimes the selected device disappears)
  • Dropdown for task result selection
  • Add timestamp of displayed result (MRD/DICOM)
  • Dicom Viewer result selection from history
  • Rearrangement of tasks by drag and drop (in template and acquisition view)
  • Added calibrations to acquisition task
  • Added a new Library view which merges sequence, device and template view
  • Download and export options

Device client improvements (WS communication):

  • Add application level heartbeat mechanism to track device status
  • Run scan callback and file upload as background tasks (non blocking)
  • Add device state machine
  • Add file upload queue
  • Allow multiple uploads within one scan callback

Data export

  • Added DICOM download
  • Added ISMRMRD download
  • Added DICOM export to XNAT database (tested with XNAT4Tests)

@schote schote added this to the Minimum Viable Product (MVP) milestone Oct 7, 2025
@schote schote self-assigned this Oct 7, 2025
@schote schote changed the title Dagster assets Workflow and device communication: Robustness and performance update Oct 10, 2025
@schote
schote requested a review from chdinh January 22, 2026 11:59
@schote
schote marked this pull request as ready for review January 22, 2026 11:59
@chdinh
chdinh merged commit 165a158 into main Feb 1, 2026
11 checks passed
@chdinh

chdinh commented Feb 1, 2026

Copy link
Copy Markdown
Member

Awesome! - Almost too large to review X-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment