View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0026445 | Community | OCCT:Modeling Data | public | 2015-07-15 17:28 | 2015-08-19 11:48 |
| Reporter | Vico Liang | Assigned To | bugmaster | ||
| Priority | normal | Severity | minor | ||
| Status | closed | Resolution | no change required | ||
| Product Version | 6.9.0 | ||||
| Summary | 0026445: It's impossible to create Ellipse with MinorRadius > MajorRadius. | ||||
| Description | OCCT don't allow the ellipse with MinorRadius longer than MajorRadius. But from mathematic parametric definition, it's very natural to have MinorRadius > MajorRadius. I'm not very clear about the background of such constraint in OCCT, is it possible to remove it without affecting other feature? | ||||
| Tags | No tags attached. | ||||
| Test case number | |||||
|
|
I had a look in the source code and found out that this limitation was made in order to make the code more simple. Let's look at the method Eccentricity: inline Standard_Real gp_Elips2d::Eccentricity() const { if (majorRadius == 0.0) { return 0.0; } else { return sqrt(majorRadius * majorRadius - minorRadius * minorRadius) / majorRadius; } } If we let majorRadius < minorRadius then we must make another branch in this method. There are many such places of code in classes describing 2D and 3D ellipse in packages gp and Geom. Probably the performance may suffer if we make such complication. It should be carefully tested. The usual solution is finding the axis of greater radius and call constructor with proper parameters. If anyway you would like us to make such improvement, please make request to our support. |
|
|
This is not a problem for me to construct an ellipse to obey the rule of OCCT. I post this issue because i'm not just thinking as a software developer. It depends on the OCCT team whether or not to make such improvement, As a user, i do hope occt become more and more powerful and user friendly. |
|
|
Andrey, let's decide if it is worth investing in this improvement. As for me, I would not invest, by 2 reasons: 1. We should find correct balance between "powerful" and "user friendly". Because frequently making a thing more user friendly makes it less powerful. 2. This improvement will require significant work load. |
|
|
I find that ACIS and Parasolid ellipse curve has such constraint too. But i do find some other system without this constraint. Dear Andrey, i agree with msv and please close this issue. |
|
|
Dear Vico Liang, thank you for understanding. Dear bugmaster, please close this bug. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2015-07-15 17:28 | Vico Liang | New Issue | |
| 2015-07-15 17:28 | Vico Liang | Assigned To | => msv |
| 2015-07-15 18:28 |
|
Note Added: 0043087 | |
| 2015-07-15 18:29 |
|
Assigned To | msv => Vico Liang |
| 2015-07-15 18:29 |
|
Status | new => feedback |
| 2015-07-16 10:40 | Vico Liang | Note Added: 0043101 | |
| 2015-07-16 10:41 | Vico Liang | Assigned To | Vico Liang => msv |
| 2015-07-16 10:50 |
|
Assigned To | msv => abv |
| 2015-07-16 10:55 |
|
Note Added: 0043103 | |
| 2015-07-16 10:55 |
|
Note Edited: 0043103 | |
| 2015-07-22 08:14 | Vico Liang | Note Added: 0043268 | |
| 2015-07-22 09:11 |
|
Note Added: 0043269 | |
| 2015-07-22 09:11 |
|
Status | feedback => reviewed |
| 2015-07-22 09:11 |
|
Resolution | open => no change required |
| 2015-07-22 09:11 |
|
Assigned To | abv => bugmaster |
| 2015-07-23 16:43 | bugmaster | Status | reviewed => closed |
| 2015-08-19 11:48 |
|
Target Version | 7.0.0 => |