Ticket #10674 (closed: fixed)
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: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