Stability of CreateSampleShape GUI

[NickDraper]

This issue was originally TRAC 3737

This ticket is blocked by :

• TRAC7802

The gui can cause Mantid to crash when insufficient information is present to display the sample.

e.g.

1. Create a sphere (enter no details)
2. Create a union above
3. Add a cylinder to the union (no details)
4. Click update 3d view -> freeze

[NickDraper]

@NickDraper (2012-01-09T09:43:02):
Moved to iteration 33 at iteration 32 code freeze

---

@NickDraper (2012-04-30T14:12:09):
Moved at end of release 2.1

---

@NickDraper (2012-08-10T12:43:26):
Moved at the end of release 2.2

---

@NickDraper (2012-10-28T11:38:40):
Moved to milestone 2.4

---

@NickDraper (2013-01-28T09:22:53):
Moved at the code freeze for release 2.4

---

@NickDraper (2013-04-29T09:49:33):
Moved to r2.6 at the end of r2.5

---

@martyngigg (2013-06-18T14:35:31):
This should also be able to load a previously created shape

---

Keith Brown (2013-07-15T09:54:05):
There seems to be two parts to this:

• Fix the crash
• Make the custom GUI operate more sensibly, populating itself if possible.

  ---

Keith Brown (2013-07-16T13:45:16):
Fixed crash by means of extra validation

The program should no longer freeze as it is no longer possible to update
the 3D view or create the shape without entering in all the details for a
component shape.
If any fields don't contain a number that's convertable to double, the
program will show an error message.
The error doesn't show what's wrong specifically, it just tells the user
there is something to check.

Refs http://trac.mantidproject.org/mantid/ticket/3737

549e777

---

Keith Brown (2013-07-17T12:21:16):
Stashing in order to fix another ticket

Validation has had a fix as in the shape valid() methods I had negated some conditions when I shouldn't have.

Otherwise, work is starting on loading a stored shape

Refs http://trac.mantidproject.org/mantid/ticket/3737

f1fd374

---

Keith Brown (2013-07-18T15:17:43):
Started writing the shape XML parser

The XML parser has been started and currently only gets the algebra string so that the tree can be constructed.

Next job is to actually build a data structure to hold the tree info in

Refs http://trac.mantidproject.org/mantid/ticket/3737

d1f6d58

---

Keith Brown (2013-07-19T09:29:42):
Started writing equation parser

Committing to save progress before looking at another branch
the equasion parser will be written as a recursive function, finding the most central opperator and splitting the string there and passing to a lower function until only a shape remains

Refs http://trac.mantidproject.org/mantid/ticket/3737

4dfa9c4

---

Keith Brown (2013-07-19T15:58:01):
Equation parser taking shape

the recursive equation parsing algorithm is taking shape and the framework is nealry complete

Refs http://trac.mantidproject.org/mantid/ticket/3737

ac1b131

---

Keith Brown (2013-07-22T13:59:18):
Parsing fucntion practically complete.

The parsing function is as complete as it can be for now, as it can't create anything before i supply the storage framework for storing the dummy tree
The new constructors for the shapes need written too

Refs http://trac.mantidproject.org/mantid/ticket/3737

1bdfcc7

---

Keith Brown (2013-08-16T11:21:43):
Discovered another bug where if a shape has it's complement flag set (the text in the widget has a # in it) it won't be able to be edited as setupDetailsBox() looks directly to see if item->text(0) is in m_setup_map, which it won't be as the # is in the way. so it continues without adding the editing widget, leaving a blank space.

Will have to change either the map or strip out the # before looking it up.

---

Keith Brown (2013-08-21T08:17:07):
Stashing to switch tickets

Now creating the functionality to populate the tree.
Committing in order to switch branches and fix a reopened ticket

Refs http://trac.mantidproject.org/mantid/ticket/3737

fc16513

---

Keith Brown (2013-08-21T08:17:08):
Instanciation methods taking shape

The instanciation methods that build the QTreeWidget are taking shape.

I've added new constructors to ShapeDetails and derived classes that take a Poco::XML::Element and can fill in details from the XML contained within that element

Refs http://trac.mantidproject.org/mantid/ticket/3737

61cb9ab

---

Keith Brown (2013-08-21T08:17:08):
Tree is being built, XML being parsed

The tree is being built as expected and the XML is being parsed, but i've revealed that the new way i've done something seems to mean that widgets aren't being loaded, but i'll fix that by making it get the widget a more sensible way.

Refs http://trac.mantidproject.org/mantid/ticket/3737

8301b67

---

Keith Brown (2013-08-21T16:04:53):
Some shapes confirmed to work. Complement identification works

At least spheres and finite cylinders import properly, the code is there for sliceofcylinderring and infinitecylinder but they've not been tested. The other shapes are awaiting parseing code.

I had made some mistakes when it came to complements of shapes. The first was that even if they were identified they weren't getting set as them, this was because I'd accidentally redeclared the flag as local and it was going out of scope. the second was that the displayed text was wrong due to the way i was constructing the QString.

Fixed a possible memory leak (or at least objects were kept in memory and not deleted) as when i was parsing XML in the shapes, some lists weren't being released.

Refs http://trac.mantidproject.org/mantid/ticket/3737

95686b7

---

Keith Brown (2013-08-22T13:22:20):
All shapes now Parse XML given to them

All shapes now have a construtor that will intialise them from XML.

I still need to inlcude parts for the spherical coordinate system but once that's in all i need to do is test it out and make sure it works fully

Refs http://trac.mantidproject.org/mantid/ticket/3737

bc1149c

---

Keith Brown (2014-06-18T07:42:24):
This is blocked as some shapes don't render properly

---

Keith Brown (2014-06-26T13:17:41):
not going to get this done before i leave, leaving to anyone

There is work against this ticket on branch bugfix/3737_CreateSampleShape_freeze_and_load_fix

[NickDraper]

not frequently used- archived