Ticket #10674 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Crash when loading project with blank matrix

Reported by: Harry Jeffery Owned by: Harry Jeffery
Priority: blocker Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Dan Nixon

Description

Just discovered this edge case.

To reproduce:

  • New project
  • New matrix
    • Don't enter any values, just leave it.
  • Save project
  • Restart mantid / new project
  • Load project with matrix
  • Crash

Change History

comment:1 Changed 6 years ago by Harry Jeffery

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Harry Jeffery

  • Status changed from assigned to inprogress

Refs #10674 Fix crash when loading blank matrix

Changeset: ef3354591daaaa4c69d5b04102e9d47c2fe5b3a0

comment:3 Changed 6 years ago by Harry Jeffery

  • Status changed from inprogress to verify
  • Resolution set to fixed

Testing

  • Follow reproduction steps in ticket description
  • Mantid should load the blank matrix successfully

comment:4 Changed 6 years ago by Dan Nixon

  • Status changed from verify to verifying
  • Tester set to Dan Nixon

comment:5 Changed 6 years ago by Dan Nixon

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/10674_crash_loading_matrix'

Full changeset: ​8114bb1750adbc5938e366b6864e2563875fc842

comment:6 Changed 6 years ago by Alex Buts

Re #10674 This should fix it

Changeset: a2f4d9286b6245cde02811ec23b375858482a7cf

comment:7 Changed 6 years ago by Alex Buts

Re #10674 Deleted unused (and not working) old parameter files

and properly modified one which used.

Changeset: f7d298f222b195efcb92cf03f4b0007e027b431d

comment:8 Changed 6 years ago by Alex Buts

Re #10674 Better formatted diagnostics

(does it have anything to do with failing unit test?)

Changeset: 147e290658b40d873f7e0be5391cd5bc282631ab

comment:9 Changed 6 years ago by Alex Buts

Re #10674 minor niceness

Changeset: 8de4bde95c47df1440e24315bd932a76cbf0a1e1

comment:10 Changed 6 years ago by Federico Montesino Pouzols

Merge remote-tracking branch 'origin/bugfix/10674_hardmaskOpt'

Full changeset: ​9952ee5b20e2e8bda18a13dbe67f7005b7d929b2

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​11516

Note: See TracTickets for help on using tickets.