Skip to content
Snippets Groups Projects

Fix serialization and plotting jobs without loaded data

Merged Mikhail Svechnikov requested to merge m.0 into main
  1. Instrument is a PolyPtr now. This fixes job deserialization.
  2. Single-purpose signals like newJobFinished and updJobView are used instead of general-purpose signals to reduce excessive calling and to ease understanding signal chain.
  3. Plot1DFrame and Plot2DFrame now have conditional subscriptions, depending on their context (data, sim or job) and react only on related events.
  4. Some connections do require flag Qt::UniqueConnection. Otherwise the number of calls accumulates and grows over time.
Edited by Mikhail Svechnikov

Merge request reports

Pipeline #173008 passed

Pipeline passed for e679be76 on m.0

Approved by

Merged by Mikhail SvechnikovMikhail Svechnikov 4 months ago (Nov 15, 2024 1:51pm UTC)

Merge details

  • Changes merged into main with 2b8d701f.
  • Deleted the source branch.

Pipeline #173391 passed

Pipeline passed for 2b8d701f on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading