Ticket #7971 (closed: fixed)
make centre optional for sphere shape
Reported by: | Anders Markvardsen | Owned by: | Peter Parker |
---|---|---|---|
Priority: | major | Milestone: | Release 3.0 |
Component: | Framework | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | #7940, #7970 | Tester: | Samuel Jackson |
Description
<sphere id="A">
<centre x="4.1" y="2.1" z="8.1" /> <radius val="3.2" />
</sphere>
make centre optional and default to (0,0,0)
Change History
comment:2 Changed 7 years ago by Peter Parker
- Status changed from inprogress to verify
- Resolution set to fixed
- Milestone changed from Backlog to Release 3.0
comment:3 Changed 7 years ago by Peter Parker
Tester:
This is a simple ticket, so shouldn't require too much testing. Make sure that you can define circle shapes in IDF files without using the centre field, and that (0,0,0) is the default value in this case. Make sure the changes look reasonable.
Instructions about IDFs for the uninitiated:
comment:4 Changed 7 years ago by Peter Parker
- Blocking 7940, 7970 added
This ticket contains getOptionalShapElement(), which is required by two other tickets.
comment:5 Changed 7 years ago by Samuel Jackson
- Status changed from verify to verifying
- Tester set to Samuel Jackson
comment:6 Changed 7 years ago by Samuel Jackson
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/7971_make_centre_optional_for_sphere_shape'
Full changeset: 8be5c4d9eb35ca250d2eea86998e25e6c1b1c968
Refs #7971 - Allow default centre for Sphere shapes.
Plus add unit test.
Changeset: 3cc0ff36dcf8f76e4baa41fb62e760bb17f25404