Sketcher: Switch to perpendicular based instead of "de la Hire" in setting minor axis of ellipse
Signed-off-by: Yash Suthar <yashsuthar983@gmail.com>
This commit is contained in:
committed by
Kacper Donat
parent
dee4ab6dbd
commit
9beca4363a
@@ -395,7 +395,7 @@ private:
|
|||||||
auto lprojy = projy.Length(); // Py = b sin t
|
auto lprojy = projy.Length(); // Py = b sin t
|
||||||
|
|
||||||
if (lprojx > firstRadius) {
|
if (lprojx > firstRadius) {
|
||||||
secondRadius = 0.0;
|
secondRadius = lprojy;
|
||||||
}
|
}
|
||||||
else {
|
else {
|
||||||
double t = std::acos(lprojx / firstRadius);
|
double t = std::acos(lprojx / firstRadius);
|
||||||
|
|||||||
Reference in New Issue
Block a user