Ticket #7802 (inprogress)
Some Shapes don't render in the CreateSampleShape GUI
Reported by: | Keith Brown | Owned by: | Roman Tolchenov |
---|---|---|---|
Priority: | major | Milestone: | Backlog |
Component: | Framework | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | #3737 | Tester: |
Description (last modified by Keith Brown) (diff)
There are some shapes in the CreateSampleShape GUI that won't render either on their own or as part of an operation.
The affected shapes are: Hexahedron Cylinder Ring Slice Cone
Hexahedrons and Cones should render and they're useful in samples and should be fixed by this ticket.
Cylinder Ring Slices aren't as useful in samples, but if they're fixed by fixing Hexahedrons and Cones, great, however if they would require specific extra work in order to get them to render, they should actually be considered to be removed as an option in the CreateSampleShape GUI as they're unlikely to be used in samples
Change History
comment:2 Changed 7 years ago by Keith Brown
- Status changed from new to inprogress
Looking for cause of non-render
Currently breakpointing the suspect area.
Refs #7802
Changeset: 84d50509afb8f57e35fc842b6882c5e73c0d40ab
comment:5 in reply to: ↑ description Changed 7 years ago by Keith Brown
- Description modified (diff)
Found out that the culprit for the cone is most likely because the bounding box is returning wrongly.
comment:6 Changed 7 years ago by Keith Brown
Added .h files to cmake list
All the primitive shape's .h files were missing from cmake. Added them even though it does nothing functionally
Refs #7802
Changeset: ffc27997eceb912227e0415cbdf6fd8deba038e9
comment:7 Changed 7 years ago by Keith Brown
Tidied up some code after my attepts to find the cause
The cause of the Cone not rendering is that it's bounding box isn't calculated and it just returns the bounds given to it. So it is actually rendering, it's jsut so small not even a pixel is small enough to represent it.
As calculating this is beyond my skill i'm going to have to leave this ticket to someone else, so i've fixed the code i'd altered in my search for the problem and tidied some other areas.
Refs #7802
Changeset: d1ab016c90b1e830456ae0342648b767c5484e93
comment:10 Changed 7 years ago by Nick Draper
- Milestone changed from Release 3.0 to Backlog
Moved out to backlog as not enough time in R3.0
comment:11 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 8647